fapifta commented on PR #4381: URL: https://github.com/apache/ozone/pull/4381#issuecomment-1466893286
Converted back the PR to draft, as I realized that the problem with the tests are not just simple glitches. In the failing tests, there are still multiple executors at the same time, and locking on the static context of the class combined with an "infinite" (Integer.MAX_VALUE) number of retries in one thread in the test causes the test to timeout whatever it takes. I will explore some other solutions, and we will discuss with @ChenSammi the original author on how we want to solve this, but the solution itself as it seems will require a deeper thought and a broader change. I don't like the simple idea of making the executor static as well, and then guard that way for the first sight, but I am open to discuss that as an option as well. We don't need to rush at the moment, as in a production system, there should not be competing threads as far as I can tell. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
