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

Set initiate_wbm_flushes to the same default value used by the WBM (true) (#480) #510

Merged
merged 3 commits into from
May 18, 2023

Conversation

udi-speedb
Copy link
Contributor

Please note that now the user of db_bench / db_stress must either override the default of initiate_wbm_flushes to false or leave it true, but set the db_write_buffer_size flag to a value > 0

@udi-speedb udi-speedb force-pushed the 480-set-initiate_wbm_flushes-to-true-in-tools branch from 6e937f9 to 44468a5 Compare May 17, 2023 06:39
@udi-speedb udi-speedb force-pushed the 480-set-initiate_wbm_flushes-to-true-in-tools branch from ef9ec63 to d522a58 Compare May 18, 2023 09:03
@udi-speedb
Copy link
Contributor Author

@Yuval-Ariel Please review my latest commit.
It reflects our last agreements - That we will create a WBM unconditionally in both db_bench and db_stress using the values of the applicable flags.

Copy link
Contributor

@Yuval-Ariel Yuval-Ariel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Yuval-Ariel Yuval-Ariel merged commit 3a8d9b9 into main May 18, 2023
@Yuval-Ariel Yuval-Ariel deleted the 480-set-initiate_wbm_flushes-to-true-in-tools branch May 18, 2023 16:21
@udi-speedb
Copy link
Contributor Author

udi-speedb commented May 21, 2023

@Yuval-Ariel I have pushed the branch 480-set-initiate_wbm_flushes-to-true-in-tools after rebase and conflict resolution
For some reason, there wasn't such a branch already on the origin

@Yuval-Ariel
Copy link
Contributor

this PR is already closed and merged 😕

udi-speedb added a commit that referenced this pull request Nov 19, 2023
* Set initiate_wbm_flushes to the same default value used by the WBM (true)

* Remove sanitizations of wbm flags in the db_bench, db_stress, and db_crashtest.py

* Create a WBM unconditionally in db_bench / db_stress (using the applicable flags)
udi-speedb added a commit that referenced this pull request Dec 5, 2023
* Set initiate_wbm_flushes to the same default value used by the WBM (true)

* Remove sanitizations of wbm flags in the db_bench, db_stress, and db_crashtest.py

* Create a WBM unconditionally in db_bench / db_stress (using the applicable flags)
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.

db_bench: Set the default of the initiate_wbm_flushes to kDfltInitiateFlushes (true) in db_bench & db_stress
2 participants