*: fix flaky test TestRecordHistoryStatsAfterAnalyze (#55365)

close #55364

pkg/executor/historical_stats_test.go

@@ -75,6 +75,7 @@ func TestRecordHistoryStatsAfterAnalyze(t *testing.T) {
 	// 3. dump current stats json
 	dumpJSONTable, err := h.DumpStatsToJSON("test", tableInfo.Meta(), nil, true)
 	require.NoError(t, err)
+	dumpJSONTable.Sort()
 	jsOrigin, _ := json.Marshal(dumpJSONTable)
 
 	// 4. get the historical stats json
@@ -89,6 +90,7 @@ func TestRecordHistoryStatsAfterAnalyze(t *testing.T) {
 	}
 	jsonTbl, err := storage.BlocksToJSONTable(data)
 	require.NoError(t, err)
+	jsonTbl.Sort()
 	jsCur, err := json.Marshal(jsonTbl)
 	require.NoError(t, err)
 	// 5. historical stats must be equal to the current stats

pkg/statistics/handle/util/util.go

@@ -15,7 +15,9 @@
 package util
 
 import (
+	"cmp"
 	"context"
+	"slices"
 	"strconv"
 	"time"
 
@@ -258,6 +260,13 @@ type JSONTable struct {
 	IsHistoricalStats bool                   `json:"is_historical_stats"`
 }
 
+// Sort is used to sort the object in the JSONTable. it is used for testing to avoid flaky test.
+func (j *JSONTable) Sort() {
+	slices.SortFunc(j.PredicateColumns, func(a, b *JSONPredicateColumn) int {
+		return cmp.Compare(a.ID, b.ID)
+	})
+}
+
 // JSONExtendedStats is used for dumping extended statistics.
 type JSONExtendedStats struct {
 	StatsName  string  `json:"stats_name"`