[ 
https://issues.apache.org/jira/browse/HDFS-9538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15056421#comment-15056421
 ] 

James Clampffer commented on HDFS-9538:
---------------------------------------

A couple more comments:

-nftw_remove calls perror to handle error conditions.  Should this have an 
#ifdef in case the user doesn't want things being printed to stderr?
-The return code for the mkdir call in TempDir::Tempdir() is never checked.  
Should this have a check that clears out path, or bails if path is already 
empty, in case of permissions issues?

> libhdfs++: load configuration from files
> ----------------------------------------
>
>                 Key: HDFS-9538
>                 URL: https://issues.apache.org/jira/browse/HDFS-9538
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: Bob Hansen
>            Assignee: Bob Hansen
>         Attachments: HDFS-9538.HDFS-9537.000.patch, 
> HDFS-9538.HDFS-9537.001.patch
>
>
> One goal of the Configuration classes are to allow the consumers of the 
> libhdfs++ library to deploy client applications into hadoop edge nodes and 
> have them pick up the Hadoop configuration that has been deployed there.
> Note that we also need to support the use case where the consumer application 
> will manage Hadoop configuration files itself, or will handle all 
> configuration out-of-band.
> libhdfs++ should be able to read files that are found in the field and easily 
> construct an instance that will communicate with the cluster.



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

Reply via email to