-
Notifications
You must be signed in to change notification settings - Fork 49
Re-change the options for rebuild index batch number #544
Re-change the options for rebuild index batch number #544
Conversation
Hold on for one sec. |
I discussed it with @critical27 , |
src/storage/StorageFlags.cpp
Outdated
@@ -26,6 +26,9 @@ DEFINE_int32(rebuild_index_batch_num, 256, | |||
DEFINE_int32(rebuild_index_locked_threshold, 256, | |||
"The locked threshold will refuse writing."); | |||
|
|||
DEFINE_int32(rebuild_index_process_interval, 30000, |
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.
3ms is enough I think? 30ms means there are 33 request at most for one second.
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.
3ms is enough I think? 30ms means there are 33 request at most for one second.
good catch. fixed.
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
I'm so sorry. the configuration file was wrong in the previous PR #540. Let's correct that in this PR.
馃槄