sashapolo commented on PR #5158: URL: https://github.com/apache/ignite-3/pull/5158#issuecomment-2655825344
Hello @12rcu , thank you for the PR. I have a couple of questions: > I changed the test annotation to a retrying test so that the test can fail once and still pass. Why would we need that? If the test is flaky, it needs to be fixed, we don't want to hide potential problems. > One source of flakiness during testing was the election timeout Could you please explain the exact reasons why this affects the flakiness? I didn't understand the explanation provided in your description. > This PR is not intended to fix the flakiness of this test per se, but to mitigate the timeout issues What timeout issues are you talking about? -- 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
