Skip to content

Commit

Permalink
some cleanup
Browse files Browse the repository at this point in the history
Signed-off-by: Vikas Choudhary (vikasc) <choudharyvikas16@gmail.com>
  • Loading branch information
vikaschoudhary16 committed Jan 15, 2025
1 parent 0966f09 commit e9690d4
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -184,8 +184,7 @@ LocalRateLimiterImpl::LocalRateLimiterImpl(
time_source_(dispatcher.timeSource()), share_provider_(std::move(shared_provider)),
always_consume_default_token_bucket_(always_consume_default_token_bucket),
no_timer_based_rate_limit_token_bucket_(Runtime::runtimeFeatureEnabled(
"envoy.reloadable_features.no_timer_based_rate_limit_token_bucket")),
dispatcher_(dispatcher) {
"envoy.reloadable_features.no_timer_based_rate_limit_token_bucket")) {
if (fill_timer_ && fill_interval < std::chrono::milliseconds(50)) {
throw EnvoyException("local rate limit token bucket fill timer must be >= 50ms");
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,6 @@ class LocalRateLimiterImpl : public Logger::Loggable<Logger::Id::rate_limit_quot
mutable Thread::ThreadSynchronizer synchronizer_; // Used for testing only.
const bool always_consume_default_token_bucket_{};
const bool no_timer_based_rate_limit_token_bucket_{};
Event::Dispatcher& dispatcher_;

friend class LocalRateLimiterImplTest;
friend class TimerTokenBucket;
Expand Down

0 comments on commit e9690d4

Please sign in to comment.