From 17aeec910e3d480f818cb6823dd4d7da44477968 Mon Sep 17 00:00:00 2001 From: rleungx Date: Wed, 13 Jan 2021 18:40:35 +0800 Subject: [PATCH] add more comments Signed-off-by: rleungx --- server/schedulers/balance_region.go | 1 + server/schedulers/balance_test.go | 1 + 2 files changed, 2 insertions(+) diff --git a/server/schedulers/balance_region.go b/server/schedulers/balance_region.go index ba3cf31b43ae..cee2746f63bd 100644 --- a/server/schedulers/balance_region.go +++ b/server/schedulers/balance_region.go @@ -58,6 +58,7 @@ const ( // balanceRegionRetryLimit is the limit to retry schedule for selected store. balanceRegionRetryLimit = 10 // BalanceEmptyRegionThreshold is a threshold which allow balance the empty region if the region number is less than this threshold. + // It is mainly used to keep the same as the original whose regions are balanced when initializing a new cluster. balanceEmptyRegionThreshold = 50 // BalanceRegionName is balance region scheduler name. BalanceRegionName = "balance-region-scheduler" diff --git a/server/schedulers/balance_test.go b/server/schedulers/balance_test.go index 509599807608..bb6255d3aa2f 100644 --- a/server/schedulers/balance_test.go +++ b/server/schedulers/balance_test.go @@ -962,6 +962,7 @@ func (s *testBalanceRegionSchedulerSuite) TestEmptyRegion(c *C) { operators := sb.Schedule(tc) c.Assert(operators, NotNil) + // exceed balanceEmptyRegionThreshold for i := uint64(10); i < 60; i++ { tc.PutRegionStores(i, 1, 3, 4) }