Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Increase the fake sleep in AdaptiveCheckTest. #1419

Merged

Conversation

lmwnshn
Copy link
Contributor

@lmwnshn lmwnshn commented Dec 27, 2020

Heading

Increase the fake sleep in AdaptiveCheckTest.

Description

Fix #1378.

I don't know why FilterManager.AdaptiveCheckTest has become more flaky nowadays.
This just builds on @gonzalezjo's previous fix in #1085 by upping the fake sleep time from 250us to 500us, so hopefully that test becomes more reliable.

I don't particularly see a point in running this through CI.

@lmwnshn lmwnshn added the ready-for-review This PR passes all checks and is ready to be reviewed. Mark PRs with this. label Dec 27, 2020
@lmwnshn lmwnshn self-assigned this Dec 27, 2020
@mbutrovich mbutrovich merged commit 208c13c into cmu-db:master Jan 5, 2021
@mbutrovich mbutrovich removed the ready-for-review This PR passes all checks and is ready to be reviewed. Mark PRs with this. label Jan 5, 2021
jkosh44 pushed a commit to jkosh44/noisepage that referenced this pull request Feb 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FilterManagerTest.AdaptiveCheckTest Failure
2 participants