dlmarion commented on PR #3230: URL: https://github.com/apache/accumulo/pull/3230#issuecomment-1460634818
> Add pre-upgrade check that looks for tserver locks in ZooKeeper and checks that the locks for the current version. Invalid locks are removed from ZooKeeper. > > * If the lock data cannot be parsed, it is removed. (data changed in 3.0) > * Make a thrift call to servers with locks that can be parsed, the thrift call will fail if incorrect version and it is removed. > For an upgrade wouldn't the typical process be? 1. Shutdown Accumulo 2. Install new version of Accumulo 3. Start Manager 4. Start remaining processes when upgrade is successful I assume then that the goal of this is to stop any TabletServer processes that are running an older version and remove their locks? I'm confused by the description I guess - is the goal to remove invalid locks or is the goal to stop processes running older versions of software and you are doing it by checking the lock data? -- 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]
