[ https://issues.apache.org/jira/browse/HDFS-2656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13453563#comment-13453563 ]
Tsz Wo (Nicholas), SZE commented on HDFS-2656: ---------------------------------------------- {quote} ... I thought this JIRA was committed rather abruptly. Since the patch was quite big (6kloc), and there had only been one or two small review comments by Nicholas above (in which he said "I just have checked the read code."), I thought the patch was still preliminary and there would be more time to review. {quote} Hi Todd, my "I just have checked the read code." was posted on Aug 20. After that Jing uploaded a new patch addressed my comment and other problems on Aug 22. Jing uploaded another patch with minor change on Sep 6. I spent a few days reviewing and testing the patch and then I +1'ed and committed it on Sep 10. Quite a few people including you provided comments on the earlier patches but no additional comments were posted. I believed there was enough time for people to review it. > Implement a pure c client based on webhdfs > ------------------------------------------ > > Key: HDFS-2656 > URL: https://issues.apache.org/jira/browse/HDFS-2656 > Project: Hadoop HDFS > Issue Type: Improvement > Components: webhdfs > Reporter: Zhanwei.Wang > Assignee: Jing Zhao > Fix For: 3.0.0 > > Attachments: HDFS-2656.patch, HDFS-2656.patch, HDFS-2656.patch, > HDFS-2656.unfinished.patch, teragen_terasort_teravalidate_performance.png > > > Currently, the implementation of libhdfs is based on JNI. The overhead of JVM > seems a little big, and libhdfs can also not be used in the environment > without hdfs. > It seems a good idea to implement a pure c client by wrapping webhdfs. It > also can be used to access different version of hdfs. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira