From 330139e38b3c6559add0f0a922961833fa2e9c35 Mon Sep 17 00:00:00 2001 From: JmPotato Date: Wed, 14 Aug 2024 12:22:36 +0800 Subject: [PATCH] tso: fix an invalid err != nil check (#8523) close tikv/pd#8524 Use the continue label to retry the whole merge loop correctly. Signed-off-by: JmPotato --- pkg/tso/keyspace_group_manager.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/pkg/tso/keyspace_group_manager.go b/pkg/tso/keyspace_group_manager.go index bfef9104086..ef3f61fec8c 100644 --- a/pkg/tso/keyspace_group_manager.go +++ b/pkg/tso/keyspace_group_manager.go @@ -1304,6 +1304,7 @@ func (kgm *KeyspaceGroupManager) mergingChecker(ctx context.Context, mergeTarget mergeMap[id] = struct{}{} } +mergeLoop: for { select { case <-ctx.Done(): @@ -1375,15 +1376,13 @@ func (kgm *KeyspaceGroupManager) mergingChecker(ctx context.Context, mergeTarget zap.Uint32("merge-id", id), zap.Time("ts", ts), zap.Error(err)) - break + // Retry from the beginning of the loop. + continue mergeLoop } if ts.After(mergedTS) { mergedTS = ts } } - if err != nil { - continue - } // Update the newly merged TSO if the merged TSO is not zero. if mergedTS != typeutil.ZeroTime { log.Info("start to set the newly merged TSO",