-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
Fix the flaky cursor persist test #10250
Fix the flaky cursor persist test #10250
Conversation
03dfab2
to
3a1d6ad
Compare
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
3a1d6ad
to
646f90b
Compare
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 as long as 34 is arbitrarily chosen and not specifically required
646f90b
to
6722caf
Compare
@littlepig2013 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: Update HISTORY.md for CompactionPri::kRoundRobin. Detailed implementation can be found in [PR10107](#10107), [PR10227](#10227), [PR10250](#10250), [PR10278](#10278), [PR10316](#10316), and [PR10341](#10341) Pull Request resolved: #10421 Reviewed By: ajkr Differential Revision: D38194070 Pulled By: littlepig2013 fbshipit-source-id: 4ce153dc0bf22cd865d09c5429955023dbc90f37
Summary:
The 'PersistRoundRobinCompactCursor' unit test in
db_compaction_test
may occasionally fail due to the inconsistent LSM state. The issue is fixed by addingFlush()
andWaitForFlushMemTable()
to produce a more predictable and stable LSM state.Test: 'PersistRoundRobinCompactCursor' unit test in
db_compaction_test