ctubbsii commented on PR #5537: URL: https://github.com/apache/accumulo/pull/5537#issuecomment-2902363655
> > You should run ScanServerUpgrade11to12TestIT.testScanRefTableCreation with this change locally. It was hanging trying to scan the root and metadata tables because they were not hosted. > > I think that is the problem that needs the balancer to run in a separate thread. With this change was just attempting to clean up some tech debt in the way the balancer code is organized before making the threading changes. Hoping we can make a change to spin up three balancer threads and remove balancing from the status thread after this change. I created #5577 to create a new IT to test this, because ScanServerUpgrade11to12TestIT.testScanRefTableCreation should be removed. I think it should be possible to create a reproducer test for that, which does not rely on removing a built-in table. -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
