[
https://issues.apache.org/jira/browse/HDFS-7014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14158854#comment-14158854
]
Colin Patrick McCabe commented on HDFS-7014:
--------------------------------------------
I did some work on this today to fix things up.
* fixed longer-than-80 characters lines
* rename cpp -> cc
* fix misspellings in error messages
* remove references to things that hdfs doesn't currently support (truncate,
etc.) We can add them back in later if HDFS gets support for those things.
* forward declare the Permission class rather than including Permission.h in
one case
* some protobuf namespace fixups
* we don't need InputStreamInter and OutputStreamInter. Since there is only
ever going to be one implementation, we can just reference InputStreamImpl and
OutputStreamImpl directly. There are a few other cases we can de-virtualize
things, such as NamenodeProxy (which doesn't need a base class). This will be
a speedup and also make the code easier to understand.
* Fix some cases where classes had implicit copy constructors that did
something dangerous. For example, copying a LocalBlockReader is not going to
end well, because the copy will refer to the same file descriptor and start
seeking around within it. This will interfere with the original. So the those
copy constructors should be disabled. Similarly, copying a FileSystem doesn't
make a lot of sense.
I still have to do a few things, will post soon! Sorry for the wait.
> Implement input and output streams to DataNode for native client
> ----------------------------------------------------------------
>
> Key: HDFS-7014
> URL: https://issues.apache.org/jira/browse/HDFS-7014
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: hdfs-client
> Reporter: Zhanwei Wang
> Assignee: Colin Patrick McCabe
> Attachments: HDFS-7014.patch
>
>
> Implement Client - Namenode RPC protocol and support Namenode HA.
> Implement Client - Datanode RPC protocol.
> Implement some basic server side class such as ExtendedBlock and LocatedBlock
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
