Manno15 opened a new pull request #2321: URL: https://github.com/apache/accumulo/pull/2321
Solution for #2184. An exception would be thrown by `HighlyAvailableServiceInvocationHandler` if the manager wasn't initialized (set by an atomic boolean inside of Manager) by the time it got a ManagerMessage from a tserver. The initialization process gets delayed due to things upgrading in the background (we wait for metadata to finish upgrading before proceeding to allow fate usage for example). This proof of concept solution adds another boolean to keep track of the manager upgrade status. So the exception will only be thrown if the manager isn't initialized due to an error instead of during an upgrade. WIP still as I am unsure on what `HighlyAvailableServiceInvocationHandler` should do if it notices manager isn't active due to upgrade. The placements of when to set and unset the new boolean could also be tweaked. -- 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]
