[GitHub] ctubbsii commented on issue #783: Configure Accumulo clients with HDFS root

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #783: Configure Accumulo clients with HDFS root
URL: https://github.com/apache/accumulo/issues/783#issuecomment-442713742
 
 
   `fs.defaultFS` is only used for qualifying unqualified paths (paths without a filesystem specified). It is probably bad practice to rely on this mechanism when writing Hadoop client code (it's too dependent on whatever Hadoop configuration is on the class path, or whichever Hadoop `Configuration` object was used to create the `Path` object).
   
   Instead of providing our own qualification mechanism (which I think we **should not** do), we should encourage users to qualify their paths with a Hadoop filesystem instead, when we receive it in our API. If they choose not to do so, and instead rely on the Hadoop behavior of automatically qualifying it with `fs.defaultFS`, then we should not override that behavior with our own. It is not our responsibility to override the natural behavior of the Hadoop client library we're using.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services