statistics: lite init used wrong value to build table stats ver (#58021) (#58055)

ti-chi-bot · web-flow · commit 71d125213608 · 2024-12-06T20:35:21.000Z
close #58020
diff --git a/pkg/statistics/handle/bootstrap.go b/pkg/statistics/handle/bootstrap.go
@@ -134,7 +134,7 @@ func (*Handle) initStatsHistograms4ChunkLite(cache statstypes.StatsCache, iter *
 		id := row.GetInt64(2)
 		ndv := row.GetInt64(3)
 		nullCount := row.GetInt64(5)
-		statsVer := row.GetInt64(7)
+		statsVer := row.GetInt64(8)
 		// All the objects in the table share the same stats version.
 		if statsVer != statistics.Version0 {
 			table.StatsVer = int(statsVer)
diff --git a/pkg/statistics/handle/handletest/handle_test.go b/pkg/statistics/handle/handletest/handle_test.go
@@ -1414,6 +1414,7 @@ func TestInitStatsLite(t *testing.T) {
 	require.NoError(t, h.InitStatsLite(context.Background()))
 	statsTbl1 := h.GetTableStats(tblInfo)
 	checkAllEvicted(t, statsTbl1)
+	require.Equal(t, int(statistics.Version2), statsTbl1.StatsVer)
 	{
 		// internal.AssertTableEqual(t, statsTbl0, statsTbl1)
 		// statsTbl0 is loaded when the cache has pseudo table.

Original file line number	Diff line number	Diff line change
`@@ -1414,6 +1414,7 @@ func TestInitStatsLite(t *testing.T) {`
`1414`	`1414`	`require.NoError(t, h.InitStatsLite(context.Background()))`
`1415`	`1415`	`statsTbl1 := h.GetTableStats(tblInfo)`
`1416`	`1416`	`checkAllEvicted(t, statsTbl1)`
	`1417`	`+ require.Equal(t, int(statistics.Version2), statsTbl1.StatsVer)`
`1417`	`1418`	`{`
`1418`	`1419`	`// internal.AssertTableEqual(t, statsTbl0, statsTbl1)`
`1419`	`1420`	`// statsTbl0 is loaded when the cache has pseudo table.`