planner: return error if binding sql is too long (pingcap#59982)

ref pingcap#51347

Author: qw4990

pkg/bindinfo/BUILD.bazel

@@ -54,7 +54,7 @@ go_test(
     embed = [":bindinfo"],
     flaky = True,
     race = "on",
-    shard_count = 32,
+    shard_count = 33,
     deps = [
         "//pkg/parser",
         "//pkg/parser/ast",

pkg/bindinfo/binding_operator.go

@@ -93,6 +93,9 @@ func (op *bindingOperator) CreateBinding(sctx sessionctx.Context, bindings []*Bi
 			binding.CreateTime = now
 			binding.UpdateTime = now
 
+			// TODO: update the sql_mode or sctx.types.Flag to let execution engine returns errors like dataTooLong,
+			// overflow directly.
+
 			// Insert the Bindings to the storage.
 			_, err = exec(
 				sctx,
@@ -118,6 +121,14 @@ func (op *bindingOperator) CreateBinding(sctx sessionctx.Context, bindings []*Bi
 			if err != nil {
 				return err
 			}
+
+			warnings, _, err := execRows(sctx, "show warnings")
+			if err != nil {
+				return err
+			}
+			if len(warnings) != 0 {
+				return errors.New(warnings[0].GetString(2))
+			}
 		}
 		return nil
 	})

pkg/bindinfo/binding_operator_test.go

@@ -17,6 +17,7 @@ package bindinfo_test
 import (
 	"context"
 	"fmt"
+	"strings"
 	"testing"
 	"time"
 
@@ -875,6 +876,21 @@ func TestBindingQueryInList(t *testing.T) {
 	}
 }
 
+func TestTooLongBinding(t *testing.T) {
+	store := testkit.CreateMockStore(t)
+	tk := testkit.NewTestKit(t, store)
+
+	tk.MustExec(`use test`)
+	tk.MustExec(`create table t (a int)`)
+	predicates := make([]string, 0, 65535)
+	for i := 0; i < cap(predicates); i++ {
+		predicates = append(predicates, fmt.Sprintf("t.a=%d", i))
+	}
+	bindingSQL := fmt.Sprintf("create global binding using select * from t where %s", strings.Join(predicates, " or "))
+	err := tk.ExecToErr(bindingSQL)
+	require.ErrorContains(t, err, "Data too long")
+}
+
 // TestBindingInListWithSingleLiteral tests sql with "IN (Lit)", fixes #44298
 func TestBindingInListWithSingleLiteral(t *testing.T) {
 	store, dom := testkit.CreateMockStoreAndDomain(t)