bbejeck commented on PR #16970: URL: https://github.com/apache/kafka/pull/16970#issuecomment-2317869299
> Not sure if I see why this would be different? If it's call because restore did not finish yet, the old and new code would pass, and if it's not called before restore was to fast, the old and new code would fail? The previous failures always involved the timing of pausing the restoration. By removing the timing of the pause I believe the test is more stable, waiting for the end of the restoration is more reliable. As you said before, this test is full of race conditions and there's no way to remove all the timing 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]
