Skip to content

Dynamic throttle metric threshold#7742

Merged
shlomi-noach merged 4 commits intovitessio:masterfrom
planetscale:dynamic-throttle-metric-threshold
Apr 22, 2021
Merged

Dynamic throttle metric threshold#7742
shlomi-noach merged 4 commits intovitessio:masterfrom
planetscale:dynamic-throttle-metric-threshold

Conversation

@shlomi-noach
Copy link
Contributor

Description

Fixes #7741

This PR adds ThrottleMetricThreshold to debugEnvHandler, meaning you can control the metrics threshold dynamically.

Related Issue(s)

Checklist

  • Should this PR be backported?
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

Impacted Areas in Vitess

Components that this PR will affect:

  • Query Serving
  • VReplication
  • Cluster Management
  • Build/CI
  • VTAdmin

Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
Signed-off-by: Shlomi Noach <2607934+shlomi-noach@users.noreply.github.com>
@shlomi-noach shlomi-noach merged commit 17c123b into vitessio:master Apr 22, 2021
@shlomi-noach shlomi-noach deleted the dynamic-throttle-metric-threshold branch April 22, 2021 13:09
@systay systay added Component: Cluster management Type: Enhancement Logical improvement (somewhere between a bug and feature) labels Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Cluster management Type: Enhancement Logical improvement (somewhere between a bug and feature)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tablet throttler: support dynamic changes to throttle_metrics_threshold

3 participants