wchevreuil commented on a change in pull request #740: HBASE-23197
'IllegalArgumentException: Wrong FS' on edits replay when…
URL: https://github.com/apache/hbase/pull/740#discussion_r337140083
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/backup/HFileArchiver.java
##########
@@ -314,9 +316,15 @@ public static void archiveStoreFiles(Configuration conf,
FileSystem fs, RegionIn
// build the archive path
if (regionInfo == null || family == null) throw new IOException(
"Need to have a region and a family to archive from.");
-
- Path storeArchiveDir = HFileArchiveUtil.getStoreArchivePath(conf,
regionInfo, tableDir, family);
-
+ String workingDir = conf.get(CommonFSUtils.HBASE_WAL_DIR);
+ Path rootDir = null;
+ if(workingDir == null || !workingDir.startsWith(fs.getScheme())){
Review comment:
> But, we could also have a wal.dir on a different path, but the same
filesystem, as the store files.
Yeah, in this case here we would pick the normal "/hbase/archive/data/..."
path as it's currently done (uses hbase.root.dir).
----------------------------------------------------------------
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]
With regards,
Apache Git Services
