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

feat: ratelimiter support auto tune #1374

Merged
merged 7 commits into from
May 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions conf/pika.conf
Original file line number Diff line number Diff line change
Expand Up @@ -326,6 +326,12 @@ max-bytes-for-level-multiplier : 10
# rate limiter bandwidth, default 200MB
#rate-limiter-bandwidth : 209715200

# rate limiter auto tune https://rocksdb.org/blog/2017/12/18/17-auto-tuned-rate-limiter.html
# default: false
#rate-limiter-refill-period-us : 100000
#rate-limiter-fairness: 10
#rate-limiter-auto-tuned : true

###########################
# rocksdb blob configure
# wiki https:/facebook/rocksdb/wiki/BlobDB
Expand Down
17 changes: 17 additions & 0 deletions include/pika_conf.h
Original file line number Diff line number Diff line change
Expand Up @@ -266,6 +266,18 @@ class PikaConf : public pstd::BaseConf {
RWLock l(&rwlock_, false);
return rate_limiter_bandwidth_;
}
int64_t rate_limiter_refill_period_us() {
RWLock l(&rwlock_, false);
return rate_limiter_refill_period_us_;
}
int64_t rate_limiter_fairness() {
RWLock l(&rwlock_, false);
return rate_limiter_fairness_;
}
bool rate_limiter_auto_tuned() {
RWLock l(&rwlock_, false);
return rate_limiter_auto_tuned_;
}

bool enable_blob_files() { return enable_blob_files_; }
int64_t min_blob_size() { return min_blob_size_; }
Expand Down Expand Up @@ -517,6 +529,7 @@ class PikaConf : public pstd::BaseConf {
int expire_logs_nums_ = 0;
bool slave_read_only_ = false;
std::string conf_path_;

int max_cache_statistic_keys_ = 0;
int small_compaction_threshold_ = 0;
int max_background_flushes_ = 0;
Expand All @@ -532,6 +545,10 @@ class PikaConf : public pstd::BaseConf {
bool optimize_filters_for_hits_ = false;
bool level_compaction_dynamic_level_bytes_ = false;
int64_t rate_limiter_bandwidth_ = 200 * 1024 * 1024; // 200M
int64_t rate_limiter_refill_period_us_ = 100 * 1000;
int64_t rate_limiter_fairness_ = 10;
bool rate_limiter_auto_tuned_ = true;

std::atomic<int> sync_window_size_;
std::atomic<int> max_conn_rbuf_size_;
std::atomic<int> consensus_level_;
Expand Down
18 changes: 18 additions & 0 deletions src/pika_admin.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1641,6 +1641,24 @@ void ConfigCmd::ConfigGet(std::string& ret) {
EncodeInt64(&config_body, g_pika_conf->rate_limiter_bandwidth());
}

if (pstd::stringmatch(pattern.data(), "rate-limiter-refill-period-us", 1)) {
elements += 2;
EncodeString(&config_body, "rate-limiter-refill-period-us");
EncodeInt64(&config_body, g_pika_conf->rate_limiter_refill_period_us());
}

if (pstd::stringmatch(pattern.data(), "rate-limiter-fairness", 1)) {
elements += 2;
EncodeString(&config_body, "rate-limiter-fairness");
EncodeInt64(&config_body, g_pika_conf->rate_limiter_fairness());
}

if (pstd::stringmatch(pattern.data(), "rate-limiter-auto-tuned", 1)) {
elements += 2;
EncodeString(&config_body, "rate-limiter-auto-tuned");
EncodeString(&config_body, g_pika_conf->rate_limiter_auto_tuned() ? "yes" : "no");
}

std::stringstream resp;
resp << "*" << std::to_string(elements) << "\r\n" << config_body;
ret = resp.str();
Expand Down
16 changes: 16 additions & 0 deletions src/pika_conf.cc
Original file line number Diff line number Diff line change
Expand Up @@ -369,6 +369,22 @@ int PikaConf::Load() {
rate_limiter_bandwidth_ = 200 * 1024 * 1024; // 200MB
}

// rate-limiter-refill-period-us
GetConfInt64("rate-limiter-refill-period-us", &rate_limiter_refill_period_us_);
if (rate_limiter_refill_period_us_ <= 0 ) {
rate_limiter_refill_period_us_ = 100 * 1000;
}

// rate-limiter-fairness
GetConfInt64("rate-limiter-fairness", &rate_limiter_fairness_);
if (rate_limiter_fairness_ <= 0 ) {
rate_limiter_fairness_ = 10;
}

std::string at;
GetConfStr("rate-limiter-auto-tuned", &at);
rate_limiter_auto_tuned_ = (at == "yes" || at.empty()) ? true : false;

// max_write_buffer_num
max_write_buffer_num_ = 2;
GetConfInt("max-write-buffer-num", &max_write_buffer_num_);
Expand Down
9 changes: 8 additions & 1 deletion src/pika_server.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1527,7 +1527,14 @@ void PikaServer::InitStorageOptions() {
}

storage_options_.options.rate_limiter =
std::shared_ptr<rocksdb::RateLimiter>(rocksdb::NewGenericRateLimiter(g_pika_conf->rate_limiter_bandwidth()));
std::shared_ptr<rocksdb::RateLimiter>(
rocksdb::NewGenericRateLimiter(
g_pika_conf->rate_limiter_bandwidth(),
g_pika_conf->rate_limiter_refill_period_us(),
g_pika_conf->rate_limiter_fairness(),
rocksdb::RateLimiter::Mode::kWritesOnly,
g_pika_conf->rate_limiter_auto_tuned()
));

// For Storage small compaction
storage_options_.statistics_max_size = g_pika_conf->max_cache_statistic_keys();
Expand Down