Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IGNITE-23927 ItDataSchemaSyncTest.checkSchemasCorrectlyRestore is flaky #4911

Merged
merged 13 commits into from
Dec 25, 2024

Conversation

JAkutenshi
Copy link
Contributor

@JAkutenshi JAkutenshi commented Dec 16, 2024

JIRA Ticket: IGNITE-23927

The goal

The main goal of the PR is to fix and enable back ItDataSchemaSyncTest.checkSchemasCorrectlyRestore test.

The reason

Tests are should be enabled and working.

The solution

The issue was in replication timeout equals 3s for all tests in the class, but it leads to ReplicationTimeoutException. The last is desired for the testExpectReplicationTimeout only, but due to default zone usage other tests may struggle over massive rebalances that leads to delays on message handling e.g. metastore events. The checkSchemasCorrectlyRestore had sometimes a little more that 3s time for a pratition to became a primary that leads to the corresponding exception. Normally there is 1min timeout.

The solution is to remove per-class timeout property and to set 3s timeout only for testExpectReplicationTimeout with cluster configuration change. Thus this test will thrown the expected exception because the 3s timeout and all other tests will be fine with 1min timeout.

@JAkutenshi JAkutenshi marked this pull request as ready for review December 25, 2024 09:21
@vldpyatkov vldpyatkov merged commit f9307af into apache:main Dec 25, 2024
1 check passed
@vldpyatkov vldpyatkov deleted the ignite-23927 branch December 25, 2024 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants