HDFS-16724. Modify patch based on comments

apache · Aug 12, 2022 · a0b33ae · a0b33ae
1 parent 92090b1
commit a0b33ae
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/...bf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/NoLocationException.java b/...bf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/NoLocationException.java
@@ -27,11 +27,7 @@ public class NoLocationException extends IOException {
 
   private static final long serialVersionUID = 1L;
 
-  public NoLocationException(String path, String className) {
-    super("Cannot find locations for " + path + " in " + className);
-  }
-
-  public NoLocationException(String message) {
-    super(message);
+  public NoLocationException(String path, Class<?> t) {
+    super("Cannot find locations for " + path + " in " + t.getSimpleName());
   }
 }
diff --git a/...fs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RouterRpcServer.java b/...fs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RouterRpcServer.java
@@ -1731,7 +1731,7 @@ protected List<RemoteLocation> getLocationsForPath(String path,
       final PathLocation location =
           this.subclusterResolver.getDestinationForPath(path);
       if (location == null) {
-        throw new NoLocationException(path, this.subclusterResolver.getClass().getSimpleName());
+        throw new NoLocationException(path, this.subclusterResolver.getClass());
       }
 
       // We may block some write operations