memory: Fix formatting of memory usage error for analyze table (#45696)

close #45690

executor/BUILD.bazel

@@ -287,6 +287,7 @@ go_test(
     srcs = [
         "adapter_test.go",
         "analyze_test.go",
+        "analyze_utils_test.go",
         "batch_point_get_test.go",
         "benchmark_test.go",
         "brie_test.go",

executor/analyze_utils.go

@@ -52,7 +52,7 @@ func getAnalyzePanicErr(r interface{}) error {
 			return errAnalyzeOOM
 		}
 		if strings.Contains(msg, memory.PanicMemoryExceedWarnMsg) {
-			return errors.Errorf(msg, errAnalyzeOOM)
+			return errors.Errorf("%s, %s", msg, errAnalyzeOOM)
 		}
 	}
 	if err, ok := r.(error); ok {

executor/analyze_utils_test.go

@@ -0,0 +1,29 @@
+// Copyright 2023 PingCAP, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package executor
+
+import (
+	"fmt"
+	"testing"
+
+	"github.com/pingcap/tidb/util/memory"
+	"github.com/stretchr/testify/require"
+)
+
+// https://github.com/pingcap/tidb/issues/45690
+func TestGetAnalyzePanicErr(t *testing.T) {
+	errMsg := fmt.Sprintf("%s", getAnalyzePanicErr(memory.PanicMemoryExceedWarnMsg))
+	require.NotContains(t, errMsg, `%!(EXTRA`)
+}