statistics: not put pseudo table of partition table into cache (#44374)

close #44308
pingcap · Jun 5, 2023 · 9bac155 · 9bac155
1 parent afdb741
commit 9bac155
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/statistics/handle/handle.go b/statistics/handle/handle.go
@@ -1005,12 +1005,18 @@ func (h *Handle) GetPartitionStats(tblInfo *model.TableInfo, pid int64, opts ...
 	if !ok {
 		tbl = statistics.PseudoTable(tblInfo)
 		tbl.PhysicalID = pid
-		h.updateStatsCache(statsCache.update([]*statistics.Table{tbl}, nil, statsCache.version))
+		if tblInfo.GetPartitionInfo() == nil || h.statsCacheLen() < 64 {
+			h.updateStatsCache(statsCache.update([]*statistics.Table{tbl}, nil, statsCache.version))
+		}
 		return tbl
 	}
 	return tbl
 }
 
+func (h *Handle) statsCacheLen() int {
+	return h.statsCache.Load().(statsCache).Len()
+}
+
 // updateStatsCache overrides the global statsCache with a new one, it may fail
 // if the global statsCache has been modified by others already.
 // Callers should add retry loop if necessary.

diff --git a/statistics/handle/statscache.go b/statistics/handle/statscache.go
@@ -83,6 +83,10 @@ type statsCache struct {
 	minorVersion uint64
 }
 
+func (sc statsCache) len() int {
+	return sc.statsCacheInner.Len()
+}
+
 func (sc statsCache) copy() statsCache {
 	newCache := statsCache{
 		version:      sc.version,