keith-turner commented on PR #5888:
URL: https://github.com/apache/accumulo/pull/5888#issuecomment-3303880089

   The output when running w/ table props looks like :
   
   ```
   $ accumulo upgrade --prepare
   2025-09-17T16:45:34,846 [util.UpgradeUtil] WARN : Saw table properties in 
site configuration : [table.durabiliy] 
   Thread 'upgrade' died.
   java.lang.IllegalStateException: Did not start upgrade preparation because 
table properties are present in site config which cause 4.0 upgrade to fail.  
Recommended action is to start cluster, set these properties at the namespace 
or table level if still needed, stop cluster, remove table properties from all 
accumulo.properties across cluster, run upgrade prepare again.
        at 
org.apache.accumulo.server.util.UpgradeUtil.execute(UpgradeUtil.java:130)
        at org.apache.accumulo.start.Main.lambda$execKeyword$0(Main.java:122)
        at java.base/java.lang.Thread.run(Thread.java:840)
   2025-09-17T16:45:34,847 [start.Main] ERROR: Thread 'upgrade' died.
   java.lang.IllegalStateException: Did not start upgrade preparation because 
table properties are present in site config which cause 4.0 upgrade to fail.  
Recommended action is to start cluster, set these properties at the namespace 
or table level if still needed, stop cluster, remove table properties from all 
accumulo.properties across cluster, run upgrade prepare again.
        at 
org.apache.accumulo.server.util.UpgradeUtil.execute(UpgradeUtil.java:130) 
~[accumulo-server-base-2.1.5-SNAPSHOT.jar:2.1.5-SNAPSHOT]
        at org.apache.accumulo.start.Main.lambda$execKeyword$0(Main.java:122) 
~[accumulo-start-2.1.5-SNAPSHOT.jar:2.1.5-SNAPSHOT]
        at java.base/java.lang.Thread.run(Thread.java:840) [?:?]
   ```
   
   Have not looked into what code is printing the exception twice, but that is 
something outside of the upgrade code.
   
   
   The updated help output look like : 
   
   ```
   $ accumulo upgrade --help
   Usage: upgrade [options]
   
     The upgrade command is intended to be used in the following way :
       1. Stop older version of accumulo
       2. Run 'accumulo upgrade --prepare' using the older version of accumulo
       3. Setup the newer version of the accumulo software
       4. Run 'accumulo upgrade --start' using the newer version of accumulo
       5. Start accumulo using the newer version and let the manager complete 
the upgrade
   
     Options:
       -h, -?, --help, -help
   
       --prepare
         prepare an older version instance for an upgrade to a newer non-bugfix 
         release. This command should be run using the older version of 
software 
         after the instance is shut down.
         Default: false
       -p, -props, --props
         Sets path to accumulo.properties.The classpath will be searched if 
this 
         property is not set
       -o
         Overrides configuration set in accumulo.properties (but NOT 
system-wide 
         config set in Zookeeper). Expected format: -o <key>=<value>
         Default: []
   
   ```
   
   The updated no args output takes a different path in the code and its output 
looks like :
   
   ```
   $ accumulo upgrade 
   Usage: upgrade [options]
   
     The upgrade command is intended to be used in the following way :
       1. Stop older version of accumulo
       2. Run 'accumulo upgrade --prepare' using the older version of accumulo
       3. Setup the newer version of the accumulo software
       4. Run 'accumulo upgrade --start' using the newer version of accumulo
       5. Start accumulo using the newer version and let the manager complete 
the upgrade
   
     Options:
       -h, -?, --help, -help
   
       --prepare
         prepare an older version instance for an upgrade to a newer non-bugfix 
         release. This command should be run using the older version of 
software 
         after the instance is shut down.
         Default: false
       -p, -props, --props
         Sets path to accumulo.properties.The classpath will be searched if 
this 
         property is not set
       -o
         Overrides configuration set in accumulo.properties (but NOT 
system-wide 
         config set in Zookeeper). Expected format: -o <key>=<value>
         Default: []
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to