VicoWu commented on a change in pull request #1764:
URL: https://github.com/apache/hbase/pull/1764#discussion_r431533385
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SecureBulkLoadManager.java
##########
@@ -382,22 +382,27 @@ public String prepareBulkLoad(final byte[] family, final
String srcPath, boolean
throw new IOException("Path does not reference a file: " + p);
}
- // Check to see if the source and target filesystems are the same
- if (!FSUtils.isSameHdfs(conf, srcFs, fs)) {
- LOG.debug("Bulk-load file " + srcPath + " is on different filesystem
than " +
+ try {
Review comment:
Overall, in this part, I just extend the exception catch domain from
previous just `fs.rename()` to the whole HDFS operation, they need obviously
the same process logic because they are all hdfs operations;
----------------------------------------------------------------
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]
