Apache9 commented on a change in pull request #2419:
URL: https://github.com/apache/hbase/pull/2419#discussion_r506945847
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/fs/HFileSystem.java
##########
@@ -82,11 +82,9 @@ public HFileSystem(Configuration conf, boolean
useHBaseChecksum)
// Create the default filesystem with checksum verification switched on.
// By default, any operation to this FilterFileSystem occurs on
// the underlying filesystem that has checksums switched on.
- this.fs = FileSystem.get(conf);
+ this.fs = FileSystem.get(getDefaultUri(conf), conf);
Review comment:
Mind explaining a bit why we need this change? And also add it to the
comment above? And I also see later we all change to use URI instead of Path.
What is the reason?
----------------------------------------------------------------
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]
