*: only output log when error is non-nil (pingcap#53350)

close pingcap#53349
terry1purcell · May 17, 2024 · 8fc7a14 · 8fc7a14
1 parent f684ddc
commit 8fc7a14
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/pkg/executor/adapter.go b/pkg/executor/adapter.go
@@ -1861,9 +1861,11 @@ func (a *ExecStmt) SummaryStmt(succ bool) {
 	planGenerator := func() (p string, h string, e any) {
 		defer func() {
 			e = recover()
-			logutil.BgLogger().Warn("fail to generate plan info",
-				zap.Stack("backtrace"),
-				zap.Any("error", e))
+			if e != nil {
+				logutil.BgLogger().Warn("fail to generate plan info",
+					zap.Stack("backtrace"),
+					zap.Any("error", e))
+			}
 		}()
 		p, h = getEncodedPlan(stmtCtx, !sessVars.InRestrictedSQL)
 		return