handle_test: fix unstable test in the TestConcurrentLoadHistTimeout (#…

…32622) (#32634) close #32580
pingcap · Sep 22, 2022 · c7f30c1 · c7f30c1
1 parent 4f28cc6
commit c7f30c1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/statistics/handle/handle_hist_test.go b/statistics/handle/handle_hist_test.go
@@ -118,13 +118,14 @@ func TestConcurrentLoadHistTimeout(t *testing.T) {
 	topn = stat.Columns[tableInfo.Columns[2].ID].TopN
 	require.Equal(t, 0, hg.Len()+topn.Num())
 	// wait for timeout task to be handled
+	oldStat := stat
 	for {
 		time.Sleep(time.Millisecond * 100)
-		if len(h.StatsLoad.TimeoutColumnsCh)+len(h.StatsLoad.NeededColumnsCh) == 0 {
+		stat = h.GetTableStats(tableInfo)
+		if stat != oldStat {
 			break
 		}
 	}
-	stat = h.GetTableStats(tableInfo)
 	hg = stat.Columns[tableInfo.Columns[2].ID].Histogram
 	topn = stat.Columns[tableInfo.Columns[2].ID].TopN
 	require.Greater(t, hg.Len()+topn.Num(), 0)