sashapolo commented on PR #5158: URL: https://github.com/apache/ignite-3/pull/5158#issuecomment-2658789345
> The retry annotation: I also don't like this annotation for the same reason, but I thought we might consider it since the timeouts for the waitForCondition() method are fixed and don't account for system load. So if the machine has a high load and fails because it ran into the timeout, it has another chance to pass the test. I would leave it up to you, I would remove it. Please remove it, this is run in a test environment, we shouldn't expect high load there. -- 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...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
