Skip to content

Commit cdf6d15

Browse files
craig[bot]kvoli
andcommitted
108054: kvserver: always log store rebalancer transfers and relocates r=andrewbaptist a=kvoli Previously, the store rebalancer would not default log range lease transfers or replica relocations, nor their estimated load impact. This made debugging imbalance related issues difficult. Bump the range relocation and range lease transfer log lines from `v(1)` to `Info`, so that they are always logged. Resolves: cockroachdb#107702 Release note: None Co-authored-by: Austen McClernon <[email protected]>
2 parents d057bda + 7f5ffa5 commit cdf6d15

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

pkg/kv/kvserver/store_rebalancer.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -655,9 +655,8 @@ func (sr *StoreRebalancer) applyRangeRebalance(
655655
voterTargets, nonVoterTargets []roachpb.ReplicationTarget,
656656
) bool {
657657
descBeforeRebalance, _ := candidateReplica.DescAndSpanConfig()
658-
log.KvDistribution.VEventf(
658+
log.KvDistribution.Infof(
659659
ctx,
660-
1,
661660
"rebalancing r%d (%s load) to better balance load: voters from %v to %v; non-voters from %v to %v",
662661
candidateReplica.GetRangeID(),
663662
candidateReplica.RangeUsageInfo().Load(),
@@ -800,9 +799,8 @@ func (sr *StoreRebalancer) chooseLeaseToTransfer(
800799
continue
801800
}
802801
if targetStore, ok := rctx.allStoresList.FindStoreByID(candidate.StoreID); ok {
803-
log.KvDistribution.VEventf(
802+
log.KvDistribution.Infof(
804803
ctx,
805-
1,
806804
"transferring lease for r%d load=%s to store s%d load=%s from local store s%d load=%s",
807805
desc.RangeID,
808806
candidateReplica.RangeUsageInfo().TransferImpact(),

0 commit comments

Comments
 (0)