storage balancer info in hive ui & sensors KIKIMR-2190 #1200
Conversation
|
⚪
|
|
⚪
|
| HIVE_CHANNEL_BALANCE_STRATEGY_HEAVIEST = 0; | ||
| HIVE_CHANNEL_BALANCE_STRATEGY_RANDOM = 1; | ||
| HIVE_CHANNEL_BALANCE_STRATEGY_WEIGHTED_RANDOM = 2; |
There was a problem hiding this comment.
This introduces compatibility issues with config versions?
There was a problem hiding this comment.
It does, but this is the PR that adds the ability to use this enum through web ui. The enum is not in any of stable releases, and I don't believe anyone has a yaml-config with it anywhere at this point.
(I changed it because it turns out C++ proto enum descriptors assume that enum values are consecutive and start with zero)
ydb/core/protos/counters_hive.proto
Outdated
| @@ -44,6 +45,7 @@ enum ECumulativeCounters { | |||
| COUNTER_SUGGESTED_SCALE_UP = 10 [(CounterOpts) = {Name: "SuggestedScaleUp"}]; | |||
| COUNTER_SUGGESTED_SCALE_DOWN = 11 [(CounterOpts) = {Name: "SuggestedScaleDown"}]; | |||
| COUNTER_STORAGE_BALANCER_EXECUTED = 12 [(CounterOpts) = {Name: "StorageBalancerExecuted"}]; | |||
| COUNTER_TABLETS_REASSIGNED = 13 [(CounterOpts) = {Name: "TabletsReassigned"}]; | |||
There was a problem hiding this comment.
TabletsReassigned quite possibly will be confused with TabletsMoved.
May be there is a more accurate name for tablet-storage groups assignment?
There was a problem hiding this comment.
Added the word 'storage' to counter name.
|
⚪
|
|
⚪
|
|
⚪
|
|
⚪
|
Changelog entry
Added storage balancer releated metrics to sensors and Hive web ui
Changelog category
Additional information
...