DomGarguilo commented on PR #5853: URL: https://github.com/apache/accumulo/pull/5853#issuecomment-3275895562
I think all issues and feedback have been resolved here as of the latest commits @ctubbsii @kevinrr888 . I made sure that these changes follow the following guidelines: * When using a `Executors.newFixedThreadPool(numThreads)` in conjunction with a `new CountDownLatch(numTasks)` ensure that `numTasks` <= `numThreads` so we are not trying to wait for more tasks than there are threads to run those tasks * If a test was already using a `Executors.newCachedThreadPool`, I left it as is. These tests were working before so no need to change them here All changed tests and ITs are passing consistently. -- 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]
