Skip to content

Commit

Permalink
[#2100] Change compaction related flags
Browse files Browse the repository at this point in the history
Summary:
Change default values for flags:
use_priority_thread_pool_for_compactions - true
allow_preempting_compactions - false

Test Plan: Jenkins

Reviewers: timur, mikhail, kannan

Reviewed By: kannan

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D7213
  • Loading branch information
spolitov committed Sep 14, 2019
1 parent 6af50b8 commit 0b543e8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/yb/rocksdb/db/db_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ DEFINE_bool(use_priority_thread_pool_for_flushes, false,
"Env thread pool with Priority::HIGH will be used.");
TAG_FLAG(use_priority_thread_pool_for_flushes, runtime);

DEFINE_bool(use_priority_thread_pool_for_compactions, false,
DEFINE_bool(use_priority_thread_pool_for_compactions, true,
"When true priority thread pool will be used for compactions, otherwise "
"Env thread pool with Priority::LOW will be used.");
TAG_FLAG(use_priority_thread_pool_for_compactions, runtime);
Expand Down
2 changes: 1 addition & 1 deletion src/yb/rocksdb/util/file_reader_writer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@

#include "yb/util/priority_thread_pool.h"

DEFINE_bool(allow_preempting_compactions, true,
DEFINE_bool(allow_preempting_compactions, false,
"Whether a compaction may be preempted in favor of another compaction with higher "
"priority");

Expand Down

0 comments on commit 0b543e8

Please sign in to comment.