ZhangChaoming commented on a change in pull request #2540:
URL: https://github.com/apache/hudi/pull/2540#discussion_r573396780



##########
File path: hudi-common/src/main/java/org/apache/hudi/common/fs/FSUtils.java
##########
@@ -80,9 +80,6 @@
   private static final PathFilter ALLOW_ALL_FILTER = file -> true;
 
   public static Configuration prepareHadoopConf(Configuration conf) {
-    conf.set("fs.hdfs.impl", 
org.apache.hadoop.hdfs.DistributedFileSystem.class.getName());

Review comment:
       @vinothchandar  On my machine, the implementation of filesyetem is 
`org.apache.hadoop.fs.viewfs.ViewFileSystem`. IMO, following user's actual 
configuration will be bertter.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to