rmdmattingly commented on code in PR #6667:
URL: https://github.com/apache/hbase/pull/6667#discussion_r1946543477
##########
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/mapreduce/MapReduceRestoreToOriginalSplitsJob.java:
##########
@@ -45,7 +45,9 @@ public void run(Path[] dirPaths, TableName[] fromTables, Path
restoreRootDir,
// We are using the files from the snapshot. We should copy them rather
than move them over
conf.setBoolean(BulkLoadHFiles.ALWAYS_COPY_FILES, true);
- FileSystem fs = FileSystem.get(conf);
+ Path backupRootDir = new Path(conf.get(RestoreJob.BACKUP_ROOT_PATH_KEY));
+
+ FileSystem fs = backupRootDir.getFileSystem(conf);
Review Comment:
This looks good to me. How does the typical `MapReduceRestoreJob` handle
this? Wondering if we should knock out a similar issue there while we're here
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
