Fix additional review comments.

apache · Oct 12, 2020 · f016b39 · f016b39
1 parent 691635e
commit f016b39
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.../network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalBlockHandler.java b/.../network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalBlockHandler.java
@@ -445,7 +445,7 @@ public MergeStatuses finalizeShuffleMerge(FinalizeShuffleMerge msg) throws IOExc
 
     @Override
     public void registerApplication(String appId, String user) {
-      throw new UnsupportedOperationException("Cannot handle shuffle block merge");
+      // No-op. Do nothing.
     }
 
     @Override
@@ -466,7 +466,7 @@ public ManagedBuffer getMergedBlockData(
 
     @Override
     public MergedBlockMeta getMergedBlockMeta(String appId, int shuffleId, int reduceId) {
-      return null;
+      throw new UnsupportedOperationException("Cannot handle shuffle block merge");
     }
 
     @Override