HDFS-17714. MockResolver#removeLocation should also remove src path. (#7313). Contributed by hfutatzhanghb.

hfutatzhanghb · web-flow · commit 26b9affe0465 · 2025-03-21T11:07:12.000+08:00
Reviewed-by: Jian Zhang &lt;keepromise@apache.org&gt;
diff --git a/hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/MockResolver.java b/hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/MockResolver.java
@@ -101,6 +101,7 @@ public boolean removeLocation(String mount, String nsId, String location) {
     final RemoteLocation remoteLocation =
         new RemoteLocation(nsId, location, mount);
     if (locationsList != null) {
+      locations.remove(mount);
       return locationsList.remove(remoteLocation);
     }
     return false;

Original file line number	Diff line number	Diff line change
`@@ -101,6 +101,7 @@ public boolean removeLocation(String mount, String nsId, String location) {`
`101`	`101`	`final RemoteLocation remoteLocation =`
`102`	`102`	`new RemoteLocation(nsId, location, mount);`
`103`	`103`	`if (locationsList != null) {`
	`104`	`+ locations.remove(mount);`
`104`	`105`	`return locationsList.remove(remoteLocation);`
`105`	`106`	`}`
`106`	`107`	`return false;`