-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries #11771
[Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries #11771
Conversation
…ate between retries - mitigates flaky tests in LockManagerTest apache#11690 and ZkSessionTest apache#11032
protected TestZKServer zks; | ||
|
||
@BeforeClass(alwaysRun = true) | ||
void setup() throws Exception { | ||
@Override | ||
protected void setup() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we remove AfterClass/BeforeClass annotations?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, TestRetrySupport doesn't change that in any way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
…ate between retries (apache#11771) * [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries - mitigates flaky tests in LockManagerTest apache#11690 and ZkSessionTest apache#11032 * [Tests] Add notes about the correct usage of TestRetrySupport (cherry picked from commit 7de9992)
There's an additional change #11778 which is needed for making the test reties work for BaseMetadataStoreTests. |
…ate between retries (apache#11771) * [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries - mitigates flaky tests in LockManagerTest apache#11690 and ZkSessionTest apache#11032 * [Tests] Add notes about the correct usage of TestRetrySupport
Motivation
Attempt to mitigate the flaky tests in LockManagerTest #11690 and ZkSessionTest #11032 which use the BaseMetadataStoreTests base class. Test retries didn't succeed since the state didn't get cleaned between tests.
The downside of better test retries is that it might suppress production issues in metadata store implementation. The flakiness might be a sign of a real problem.
Modifications