From 9aaa93e9ccd901f13dbf9e09b2c989ecdf29ae6a Mon Sep 17 00:00:00 2001
From: xiongjiwei <xiongjiwei1996@outlook.com>
Date: Tue, 3 Jan 2023 19:04:19 +0900
Subject: [PATCH] test: fix data race in cast as array (#40277)

close pingcap/tidb#40276
---
 expression/builtin_cast.go | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/expression/builtin_cast.go b/expression/builtin_cast.go
index 545abd497a2da..b6fc16cda2ef4 100644
--- a/expression/builtin_cast.go
+++ b/expression/builtin_cast.go
@@ -458,6 +458,9 @@ func (b *castJSONAsArrayFunctionSig) Clone() builtinFunc {
 	return newSig
 }
 
+// fakeSctx is used to ignore the sql mode, `cast as array` should always return error if any.
+var fakeSctx = &stmtctx.StatementContext{InInsertStmt: true}
+
 func (b *castJSONAsArrayFunctionSig) evalJSON(row chunk.Row) (res types.BinaryJSON, isNull bool, err error) {
 	val, isNull, err := b.args[0].EvalJSON(b.ctx, row)
 	if isNull || err != nil {
@@ -474,20 +477,8 @@ func (b *castJSONAsArrayFunctionSig) evalJSON(row chunk.Row) (res types.BinaryJS
 	if f == nil {
 		return types.BinaryJSON{}, false, ErrNotSupportedYet.GenWithStackByArgs("CAS-ing JSON to the target type")
 	}
-	sc := b.ctx.GetSessionVars().StmtCtx
-	originalOverflowAsWarning := sc.OverflowAsWarning
-	originIgnoreTruncate := sc.IgnoreTruncate
-	originTruncateAsWarning := sc.TruncateAsWarning
-	sc.OverflowAsWarning = false
-	sc.IgnoreTruncate = false
-	sc.TruncateAsWarning = false
-	defer func() {
-		sc.OverflowAsWarning = originalOverflowAsWarning
-		sc.IgnoreTruncate = originIgnoreTruncate
-		sc.TruncateAsWarning = originTruncateAsWarning
-	}()
 	for i := 0; i < val.GetElemCount(); i++ {
-		item, err := f(sc, val.ArrayGetElem(i), ft)
+		item, err := f(fakeSctx, val.ArrayGetElem(i), ft)
 		if err != nil {
 			return types.BinaryJSON{}, false, err
 		}