Skip to content

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

Merged
Yuval-Ariel merged 3 commits into
mainfrom
480-set-initiate_wbm_flushes-to-true-in-tools
May 18, 2023
Merged

Set initiate_wbm_flushes to the same default value used by the WBM (true) (#480)#510
Yuval-Ariel merged 3 commits into
mainfrom
480-set-initiate_wbm_flushes-to-true-in-tools

Conversation

@udi-speedb

Copy link
Copy Markdown
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 requested a review from Yuval-Ariel May 17, 2023 05:28
@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
Comment thread db_stress_tool/db_stress_gflags.cc
@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
Copy Markdown
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.

@Yuval-Ariel Yuval-Ariel left a comment

Copy link
Copy Markdown
Collaborator

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

udi-speedb commented May 21, 2023

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown
Collaborator

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