Files
tasks/common/persistence/client/health_request_rate_limiter.go
David ReissandGitHub b50a12fc7c Use typed dynamic configs (#5962)
## What changed?
Use typed dynamic configs where we currently use maps that are converted
to structs.

## Why?
Simplify code, remove boilerplate.

## How did you test it?
existing tests

## Potential risks
1. If the conversion provided by mapstructure doesn't do the exact same
thing as the hand-written conversion code, some values may change
meaning.
2. In a couple cases I deliberately changed the semantics to match what
seemed to be intended behavior (noted in PR comments).
2024-06-03 18:16:15 -07:00

184 lines
6.3 KiB
Go

// The MIT License
//
// Copyright (c) 2020 Temporal Technologies Inc. All rights reserved.
//
// Copyright (c) 2020 Uber Technologies, Inc.
//
// Permission is hereby granted, free of charge, to any person obtaining a copy
// of this software and associated documentation files (the "Software"), to deal
// in the Software without restriction, including without limitation the rights
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
// copies of the Software, and to permit persons to whom the Software is
// furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included in
// all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
package client
import (
"context"
"math"
"sync/atomic"
"time"
"go.temporal.io/server/common/dynamicconfig"
"go.temporal.io/server/common/log"
"go.temporal.io/server/common/log/tag"
"go.temporal.io/server/common/metrics"
"go.temporal.io/server/common/persistence"
"go.temporal.io/server/common/quotas"
)
const (
DefaultRefreshInterval = 10 * time.Second
DefaultRateBurstRatio = 1.0
DefaultInitialRateMultiplier = 1.0
)
type (
HealthRequestRateLimiterImpl struct {
enabled atomic.Bool
params DynamicRateLimitingParams // dynamic config struct
curOptions atomic.Pointer[dynamicconfig.DynamicRateLimitingParams] // current dynamic config values (updated on refresh)
rateLimiter *quotas.RateLimiterImpl
healthSignals persistence.HealthSignalAggregator
refreshTimer *time.Ticker
rateBurst quotas.RateBurst
curRateMultiplier atomic.Pointer[float64]
metricsHandler metrics.Handler
logger log.Logger
}
)
var _ quotas.RequestRateLimiter = (*HealthRequestRateLimiterImpl)(nil)
func NewHealthRequestRateLimiterImpl(
healthSignals persistence.HealthSignalAggregator,
rateFn quotas.RateFn,
params DynamicRateLimitingParams,
burstRatio PersistenceBurstRatio,
metricsHandler metrics.Handler,
logger log.Logger,
) *HealthRequestRateLimiterImpl {
rateBurst := quotas.NewDefaultRateBurst(rateFn, quotas.BurstRatioFn(burstRatio))
limiter := &HealthRequestRateLimiterImpl{
enabled: atomic.Bool{},
rateLimiter: quotas.NewRateLimiter(rateBurst.Rate(), rateBurst.Burst()),
healthSignals: healthSignals,
rateBurst: rateBurst,
params: params,
refreshTimer: time.NewTicker(DefaultRefreshInterval),
metricsHandler: metricsHandler,
logger: logger,
}
curRateMultiplier := new(float64)
*curRateMultiplier = DefaultInitialRateMultiplier
limiter.curRateMultiplier.Store(curRateMultiplier)
limiter.refreshDynamicParams()
return limiter
}
func (rl *HealthRequestRateLimiterImpl) Allow(now time.Time, request quotas.Request) bool {
rl.maybeRefresh()
if !rl.enabled.Load() {
return true
}
return rl.rateLimiter.AllowN(now, request.Token)
}
func (rl *HealthRequestRateLimiterImpl) Reserve(now time.Time, request quotas.Request) quotas.Reservation {
rl.maybeRefresh()
if !rl.enabled.Load() {
return quotas.NoopReservation
}
return rl.rateLimiter.ReserveN(now, request.Token)
}
func (rl *HealthRequestRateLimiterImpl) Wait(ctx context.Context, request quotas.Request) error {
rl.maybeRefresh()
if !rl.enabled.Load() {
return nil
}
return rl.rateLimiter.WaitN(ctx, request.Token)
}
func (rl *HealthRequestRateLimiterImpl) maybeRefresh() {
select {
case <-rl.refreshTimer.C:
rl.refreshDynamicParams()
if rl.enabled.Load() {
rl.refreshRate()
}
rl.updateRefreshTimer()
default:
// no-op
}
}
func (rl *HealthRequestRateLimiterImpl) refreshRate() {
curOptions := *rl.curOptions.Load()
curRateMultiplier := *rl.curRateMultiplier.Load()
if rl.latencyThresholdExceeded() || rl.errorThresholdExceeded() {
// limit exceeded, do backoff
curRateMultiplier = math.Max(curOptions.RateMultiMin, curRateMultiplier-curOptions.RateBackoffStepSize)
metrics.DynamicRateLimiterMultiplier.With(rl.metricsHandler).Record(curRateMultiplier)
rl.logger.Info(
"Health threshold exceeded, reducing rate limit.",
tag.NewFloat64("newMulti", curRateMultiplier),
tag.NewFloat64("newRate", rl.rateLimiter.Rate()),
tag.NewFloat64("latencyAvg", rl.healthSignals.AverageLatency()),
tag.NewFloat64("errorRatio", rl.healthSignals.ErrorRatio()),
)
} else if curRateMultiplier < curOptions.RateMultiMax {
// already doing backoff and under thresholds, increase limit
curRateMultiplier = math.Min(curOptions.RateMultiMax, curRateMultiplier+curOptions.RateIncreaseStepSize)
metrics.DynamicRateLimiterMultiplier.With(rl.metricsHandler).Record(curRateMultiplier)
rl.logger.Info(
"System healthy, increasing rate limit.",
tag.NewFloat64("newMulti", curRateMultiplier),
tag.NewFloat64("newRate", rl.rateLimiter.Rate()),
tag.NewFloat64("latencyAvg", rl.healthSignals.AverageLatency()),
tag.NewFloat64("errorRatio", rl.healthSignals.ErrorRatio()),
)
}
rl.curRateMultiplier.Store(&curRateMultiplier)
// Always set rate to pickup changes to underlying rate limit dynamic config
rl.rateLimiter.SetRPS(curRateMultiplier * rl.rateBurst.Rate())
rl.rateLimiter.SetBurst(int(curRateMultiplier * float64(rl.rateBurst.Burst())))
}
func (rl *HealthRequestRateLimiterImpl) refreshDynamicParams() {
options := rl.params()
rl.enabled.Store(options.Enabled)
rl.curOptions.Store(&options)
}
func (rl *HealthRequestRateLimiterImpl) updateRefreshTimer() {
rl.refreshTimer.Reset(rl.curOptions.Load().RefreshInterval)
}
func (rl *HealthRequestRateLimiterImpl) latencyThresholdExceeded() bool {
curOptions := *rl.curOptions.Load()
return curOptions.LatencyThreshold > 0 && rl.healthSignals.AverageLatency() > curOptions.LatencyThreshold
}
func (rl *HealthRequestRateLimiterImpl) errorThresholdExceeded() bool {
curOptions := *rl.curOptions.Load()
return curOptions.ErrorThreshold > 0 && rl.healthSignals.ErrorRatio() > curOptions.ErrorThreshold
}