(accumulo) branch main updated: Recent change to Upgrader12to13 ate all Exceptions (#5409)

[dlmarion]

This is an automated email from the ASF dual-hosted git repository.

dlmarion pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new d1b2e895fe Recent change to Upgrader12to13 ate all Exceptions (#5409)
d1b2e895fe is described below

commit d1b2e895fe7bb993ff38553523dc7a906d53156d
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Tue Mar 18 08:02:06 2025 -0400

    Recent change to Upgrader12to13 ate all Exceptions (#5409)
---
 .../java/org/apache/accumulo/manager/upgrade/Upgrader12to13.java    | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader12to13.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader12to13.java
index 2c53c9f600..3c53bff5ba 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader12to13.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader12to13.java
@@ -137,9 +137,13 @@ public class Upgrader12to13 implements Upgrader {
     try {
       ZooKeeperInitializer zkInit = new ZooKeeperInitializer();
       zkInit.initFateTableState(context);
-    } catch (Exception e) {
+    } catch (RuntimeException e) {
+      // initFateTableState wraps KeeperException and InterruptedException
+      // with a RuntimeException
       if (e.getCause() instanceof KeeperException.NodeExistsException) {
         LOG.debug("Fate table node already exists in ZooKeeper");
+      } else {
+        throw e;
       }
     }