Rewrite some loops to use range.

pingcap · Nov 27, 2024 · 5368a64 · 5368a64
1 parent 4719db2
commit 5368a64
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/pkg/util/workloadrepo/snapshot.go b/pkg/util/workloadrepo/snapshot.go
@@ -154,7 +154,7 @@ func (w *worker) startSnapshot(_ctx context.Context) func() {
 					continue
 				}
 
-				for i := 0; i < 5; i++ {
+				for range 5 {
 					snapID, err := w.getSnapID(ctx)
 					if err != nil {
 						logutil.BgLogger().Info("workload repository cannot get current snapid", zap.NamedError("err", err))

diff --git a/pkg/util/workloadrepo/table.go b/pkg/util/workloadrepo/table.go
@@ -168,7 +168,7 @@ func checkTableExistsByIS(ctx context.Context, is infoschema.InfoSchema, tblName
 	}
 
 	tbInfo := tbSchema.Meta()
-	for i := 0; i < 2; i++ {
+	for i := range 2 {
 		newPtTime := now.AddDate(0, 0, i+1)
 		newPtName := "p" + newPtTime.Format("20060102")
 		ptInfos := tbInfo.GetPartitionInfo().Definitions

diff --git a/pkg/util/workloadrepo/utils.go b/pkg/util/workloadrepo/utils.go
@@ -38,7 +38,7 @@ func generatePartitionRanges(sb *strings.Builder, tbInfo *model.TableInfo) bool
 	// add new partitions per day
 	// if all partitions to be added existed, do nothing
 	allExisted := true
-	for i := 0; i < newPtNum; i++ {
+	for i := range newPtNum {
 		// TODO: should we make this UTC? timezone issues
 		newPtTime := now.AddDate(0, 0, i+1)
 		newPtName := "p" + newPtTime.Format("20060102")