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

Remove deprecated Options::access_hint_on_compaction_start #11654

Closed

Conversation

hx235
Copy link
Contributor

@hx235 hx235 commented Jul 29, 2023

Context:
Options::access_hint_on_compaction_start is marked deprecated and now ready to be removed.

Test plan:
Multiple db_stress runs with pre-PR and post-PR binary randomly to ensure forward/backward compatibility on options ajkr@36a5686?fbclid=IwAR2IcdAUdTvw9O9V5GkHEYJRGMVR9p7Ei-LMa-9qiXlj3z80DxjkxlGnP1E
python3 tools/db_crashtest.py --simple blackbox --interval=30

@hx235 hx235 changed the title Remove deprecated Options::access_hint_on_compaction_start [Draft] Remove deprecated Options::access_hint_on_compaction_start Jul 29, 2023
@hx235 hx235 added the WIP Work in progress label Jul 29, 2023
@facebook-github-bot
Copy link
Contributor

@hx235 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@hx235 has updated the pull request. You must reimport the pull request before landing.

@hx235 hx235 force-pushed the dep_access_hint_on_compaction_start branch 2 times, most recently from de92d4b to 5e1dcd4 Compare August 2, 2023 16:56
@facebook-github-bot
Copy link
Contributor

@hx235 has updated the pull request. You must reimport the pull request before landing.

@hx235 hx235 force-pushed the dep_access_hint_on_compaction_start branch from 5e1dcd4 to 1af0316 Compare February 2, 2024 01:24
@facebook-github-bot
Copy link
Contributor

@hx235 has updated the pull request. You must reimport the pull request before landing.

@hx235 hx235 force-pushed the dep_access_hint_on_compaction_start branch from 1af0316 to ad8386b Compare February 3, 2024 03:59
@facebook-github-bot
Copy link
Contributor

@hx235 has updated the pull request. You must reimport the pull request before landing.

@hx235 hx235 changed the title [Draft] Remove deprecated Options::access_hint_on_compaction_start Remove deprecated Options::access_hint_on_compaction_start Feb 3, 2024
@hx235 hx235 force-pushed the dep_access_hint_on_compaction_start branch from ad8386b to 6aef249 Compare February 3, 2024 04:00
@facebook-github-bot
Copy link
Contributor

@hx235 has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@hx235 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@hx235 merged this pull request in 1a885fe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants