keith-turner commented on PR #5531: URL: https://github.com/apache/accumulo/pull/5531#issuecomment-2847858235
> Meanwhile, the root tablet can't be assigned a location because the Manager.tserverStatus map is not populated, which is done from the StatusThread as well. Seen problems with the way this code works before. Would probably be best to move balancing out of the status thread and give its own thread. Could have a balancing thread for each data level, with this setup each balancing thread would only read the migrations for its level which would avoid trying to read all at once. Also would be good to minimize the dependencies between balancing and TGW as much as possible. Because of fundamental problems in the existing code the fix in this PR may avoid the problem in some situations, but the stats thread could still get stuck if things changes after its done its checks. -- 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]
