fix

Signed-off-by: Weizhen Wang <[email protected]>

Author: hawkingrei

pkg/statistics/handle/syncload/BUILD.bazel

@@ -14,6 +14,7 @@ go_library(
         "//pkg/parser/mysql",
         "//pkg/sessionctx",
         "//pkg/sessionctx/stmtctx",
+        "//pkg/sessionctx/variable",
         "//pkg/statistics",
         "//pkg/statistics/handle/storage",
         "//pkg/statistics/handle/types",

pkg/statistics/handle/syncload/stats_syncload.go

@@ -307,11 +307,14 @@ func (s *statsSyncLoad) handleOneItemTask(task *statstypes.NeededItemTask) (err
 			s.statsHandle.SPool().Destroy(se)
 		}
 	}()
+	var skipTypes map[string]struct{}
 	val, err := sctx.GetSessionVars().GlobalVarsAccessor.GetGlobalSysVar(variable.TiDBAnalyzeSkipColumnTypes)
 	if err != nil {
 		logutil.BgLogger().Warn("failed to get global variable", zap.Error(err))
+	} else {
+		skipTypes = variable.ParseAnalyzeSkipColumnTypes(val)
 	}
-	skipTypes := variable.ParseAnalyzeSkipColumnTypes(val)
+
 	item := task.Item.TableItemID
 	statsTbl, ok := s.statsHandle.Get(item.TableID)
 
@@ -348,10 +351,13 @@ func (s *statsSyncLoad) handleOneItemTask(task *statstypes.NeededItemTask) (err
 			// so we have to get the column info from the domain.
 			wrapper.colInfo = tblInfo.GetColumnByID(item.ID)
 		}
-		_, skip := skipTypes[types.TypeToStr(wrapper.colInfo.FieldType.GetType(), wrapper.colInfo.FieldType.GetCharset())]
-		if skip {
-			return nil
+		if skipTypes != nil {
+			_, skip := skipTypes[types.TypeToStr(wrapper.colInfo.FieldType.GetType(), wrapper.colInfo.FieldType.GetCharset())]
+			if skip {
+				return nil
+			}
 		}
+
 		// If this column is not analyzed yet and we don't have it in memory.
 		// We create a fake one for the pseudo estimation.
 		// Otherwise, it will trigger the sync/async load again, even if the column has not been analyzed.