HDDS-7068: EC: Prematurely re-throwed the exception in reconstruction…

… cleanup loop. (#3639)
apache · Jul 29, 2022 · 57d620d · 57d620d
1 parent daf2578
commit 57d620d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/...java/org/apache/hadoop/ozone/container/ec/reconstruction/ECReconstructionCoordinator.java b/...java/org/apache/hadoop/ozone/container/ec/reconstruction/ECReconstructionCoordinator.java
@@ -179,8 +179,8 @@ public void reconstructECContainerGroup(long containerID,
           LOG.error("Exception while deleting the container {} at target: {}",
               containerID, dn, ioe);
         }
-        throw e;
       }
+      throw e;
     }
 
   }

diff --git a/.../org/apache/hadoop/ozone/container/ec/reconstruction/ECReconstructionCoordinatorTask.java b/.../org/apache/hadoop/ozone/container/ec/reconstruction/ECReconstructionCoordinatorTask.java
@@ -64,6 +64,10 @@ public void run() {
     // respective container. HDDS-6582
     // 5. Close/finalize the recovered containers.
     long containerID = this.reconstructionCommandInfo.getContainerID();
+    if (LOG.isDebugEnabled()) {
+      LOG.debug("Starting the EC reconstruction of the container {}",
+          containerID);
+    }
     try {
       SortedMap<Integer, DatanodeDetails> sourceNodeMap =
           reconstructionCommandInfo.getSources().stream().collect(Collectors