statistics: get right max table id when to init stats

Signed-off-by: Weizhen Wang <[email protected]>
pingcap · Dec 16, 2024 · 145de7f · 145de7f
1 parent cea46f1
commit 145de7f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/pkg/statistics/handle/bootstrap.go b/pkg/statistics/handle/bootstrap.go
@@ -51,6 +51,11 @@ const (
 
 var maxTidRecord MaxTidRecord
 
+// GetMaxTidRecordForTest gets the max tid record for test.
+func GetMaxTidRecordForTest() int64 {
+	return maxTidRecord.tid.Load()
+}
+
 // MaxTidRecord is to record the max tid.
 type MaxTidRecord struct {
 	mu  sync.Mutex
@@ -84,7 +89,7 @@ func (*Handle) initStatsMeta4Chunk(cache statstypes.StatsCache, iter *chunk.Iter
 	maxTidRecord.mu.Lock()
 	defer maxTidRecord.mu.Unlock()
 	if maxTidRecord.tid.Load() < maxPhysicalID {
-		maxTidRecord.tid.Store(physicalID)
+		maxTidRecord.tid.Store(maxPhysicalID)
 	}
 }
 

diff --git a/pkg/statistics/handle/handletest/initstats/load_stats_test.go b/pkg/statistics/handle/handletest/initstats/load_stats_test.go
@@ -100,4 +100,5 @@ func testConcurrentlyInitStats(t *testing.T) {
 			require.False(t, col.IsAllEvicted())
 		}
 	}
+	require.Equal(t, int64(126), handle.GetMaxTidRecordForTest())
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -100,4 +100,5 @@ func testConcurrentlyInitStats(t *testing.T) { @@
     			require.False(t, col.IsAllEvicted())
     		}
     	}
+    	require.Equal(t, int64(126), handle.GetMaxTidRecordForTest())
     }