(accumulo) branch main updated: Add log message to UpgradeCoordinator to note when Upgrader complete (#5408)

[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 4cba6dee3e Add log message to UpgradeCoordinator to note when Upgrader 
complete (#5408)
4cba6dee3e is described below

commit 4cba6dee3e89960bbaac2f9c86eb34a3a0426779
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Mon Mar 17 14:56:22 2025 -0400

    Add log message to UpgradeCoordinator to note when Upgrader complete (#5408)
---
 .../accumulo/manager/upgrade/UpgradeCoordinator.java     | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java
index b1676b2b0d..0769a2a650 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java
@@ -211,10 +211,12 @@ public class UpgradeCoordinator {
           // if the current version is 10, then the code above will get the 
Upgrader for version
           // 10 from the upgraders map. It will run the Upgrader10to11 code, 
which means that the
           // current version of ZooKeeper after running the upgrade is 11.
-          progressTracker.updateZooKeeperVersion(upgradeVersion + 1);
+          final int nextVersion = upgradeVersion + 1;
+          progressTracker.updateZooKeeperVersion(nextVersion);
+          log.info("ZooKeeper upgrade from version {} to version {} 
complete.", upgradeVersion,
+              nextVersion);
         }
       }
-
       setStatus(UpgradeStatus.UPGRADED_ZOOKEEPER, eventCoordinator);
     } catch (Exception e) {
       handleFailure(e);
@@ -255,7 +257,10 @@ public class UpgradeCoordinator {
                 // if the current version is 10, then the code above will get 
the Upgrader for
                 // version 10 from the upgraders map. It will run the 
Upgrader10to11 code, which
                 // means that the current version of ZooKeeper after running 
the upgrade is 11.
-                progressTracker.updateRootVersion(upgradeVersion + 1);
+                final int nextVersion = upgradeVersion + 1;
+                progressTracker.updateRootVersion(nextVersion);
+                log.info("Root upgrade from version {} to version {} 
complete.", upgradeVersion,
+                    nextVersion);
               }
               setStatus(UpgradeStatus.UPGRADED_ROOT, eventCoordinator);
 
@@ -278,7 +283,10 @@ public class UpgradeCoordinator {
                 // if the current version is 10, then the code above will get 
the Upgrader for
                 // version 10 from the upgraders map. It will run the 
Upgrader10to11 code, which
                 // means that the current version of ZooKeeper after running 
the upgrade is 11.
-                progressTracker.updateMetadataVersion(upgradeVersion + 1);
+                final int nextVersion = upgradeVersion + 1;
+                progressTracker.updateMetadataVersion(nextVersion);
+                log.info("Metadata upgrade from version {} to version {} 
complete.", upgradeVersion,
+                    nextVersion);
               }
               setStatus(UpgradeStatus.UPGRADED_METADATA, eventCoordinator);