Skip to content

Commit

Permalink
tx_throttler: delete topo watcher metric instead of deprecating (vite…
Browse files Browse the repository at this point in the history
…ssio#14445)

Signed-off-by: deepthi <deepthi@planetscale.com>
  • Loading branch information
deepthi authored and timvaillancourt committed May 16, 2024
1 parent 65333f6 commit 97d455e
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions go/vt/vttablet/tabletserver/txthrottler/tx_throttler.go
Original file line number Diff line number Diff line change
Expand Up @@ -146,9 +146,7 @@ type txThrottler struct {
topoServer *topo.Server

// stats
throttlerRunning *stats.Gauge
// TODO(deepthi): deprecated, should be deleted in v20
topoWatchers *stats.GaugesWithSingleLabel
throttlerRunning *stats.Gauge
healthChecksReadTotal *stats.CountersWithMultiLabels
healthChecksRecordedTotal *stats.CountersWithMultiLabels
requestsTotal *stats.CountersWithSingleLabel
Expand Down Expand Up @@ -201,7 +199,6 @@ func NewTxThrottler(env tabletenv.Env, topoServer *topo.Server) TxThrottler {
config: config,
topoServer: topoServer,
throttlerRunning: env.Exporter().NewGauge(TxThrottlerName+"Running", "transaction throttler running state"),
topoWatchers: env.Exporter().NewGaugesWithSingleLabel(TxThrottlerName+"TopoWatchers", "DEPRECATED: transaction throttler topology watchers", "cell"),
healthChecksReadTotal: env.Exporter().NewCountersWithMultiLabels(TxThrottlerName+"HealthchecksRead", "transaction throttler healthchecks read",
[]string{"cell", "DbType"}),
healthChecksRecordedTotal: env.Exporter().NewCountersWithMultiLabels(TxThrottlerName+"HealthchecksRecorded", "transaction throttler healthchecks recorded",
Expand Down

0 comments on commit 97d455e

Please sign in to comment.