You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-dev@hadoop.apache.org by "James Clampffer (JIRA)" <ji...@apache.org> on 2016/01/27 22:36:39 UTC

[jira] [Created] (HDFS-9712) libhdfs++: Reimplement Status object as a normal struct

James Clampffer created HDFS-9712:
-------------------------------------

             Summary: libhdfs++: Reimplement Status object as a normal struct
                 Key: HDFS-9712
                 URL: https://issues.apache.org/jira/browse/HDFS-9712
             Project: Hadoop HDFS
          Issue Type: Sub-task
            Reporter: James Clampffer
            Assignee: James Clampffer


hdfs::Status is doing all sorts of reinterpret casts on a block of memory referenced by a char *.  Using a char *, casting to a wider type, and dereferencing can cause fun alignment issues.

As far as I can tell that data layout in status can be boiled down to:
{code}
class Status {
  int code;
  std::string msg;
}
{code}

This avoids doing manual memcopies in the copy ctor and delete[]s in the dtor.  It will also get rid of boilerplate null checks and casts.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)