Yiran-wu commented on code in PR #6856:
URL: https://github.com/apache/hbase/pull/6856#discussion_r2048727731


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/RecoveredReplicationSource.java:
##########
@@ -104,7 +106,30 @@ public void locateRecoveredPaths(String walGroupId) throws 
IOException {
         // didn't find a new location
         LOG.error(
           String.format("WAL Path %s doesn't exist and couldn't find its new 
location", path));
-        newPaths.add(path);
+        Path walPath = path;
+        if (

Review Comment:
   When we restore, the Path from Zookeeper(or other storage). The Zookeeper 
path does not contain the filesystem path. Judging from the code, it might be 
to reduce the complexity of modifying Zookeeper when moving files to oldWALs.



-- 
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]

Reply via email to