keith-turner opened a new pull request, #5734: URL: https://github.com/apache/accumulo/pull/5734
When a tablet server is gracefully shutdown it notifies the manager it is shutting down. The manager would remove the server from the set of live tservers which caused the manger to treat all tablets on that tserver as assigned to dead tservers. Saw this when debugging a failure with GracefulShutdownIT. Saw the manager initiate log recovery and reassignment for the root tablet before it unloaded from the tserver that was shutting down. This caused errors with the metadata tablet consistency check during unload of the root tablet because the manager had modified the tablet metadata before unload. This fix removes the notification to the manager and replaces it with a tablet server check to ignore loading tablets during tserver shutdown. Created the new OpeningTablets class because the two sets it contains are very tightly coupled, so seemed best to create a new class rather than add those two variables to TabletServer class. -- 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]
