Fixing static check problems.

Author: fengttt

pkg/frontend/starlark_interpreter.go

@@ -25,15 +25,14 @@ import (
 	ujson "github.com/matrixorigin/matrixone/pkg/util/json"
 	"go.starlark.net/starlark"
 	"go.starlark.net/starlarkstruct"
+	"go.starlark.net/syntax"
 	"go.uber.org/zap"
 )
 
 type starlarkInterpreter struct {
 	interp      *Interpreter
 	thread      *starlark.Thread
 	predeclared starlark.StringDict
-	outVars     map[string]interface{}
-	fn          starlark.Value
 }
 
 func convertToStarlarkValue(ctx context.Context, v any) (starlark.Value, error) {
@@ -213,7 +212,18 @@ func (interpreter *Interpreter) ExecuteStarlark(spBody string, dbName string, bg
 		}
 	}
 
-	globals, err := starlark.ExecFile(si.thread, "", spBody, si.predeclared)
+	globals, err := starlark.ExecFileOptions(
+		&syntax.FileOptions{
+			While:           true,
+			TopLevelControl: true,
+			GlobalReassign:  true,
+			Recursion:       true,
+		},
+		si.thread,
+		"",
+		spBody,
+		si.predeclared,
+	)
 	if err != nil {
 		return err
 	}

pkg/sql/plan/function/func_builtin_starlark.go

@@ -22,6 +22,7 @@ import (
 	ujson "github.com/matrixorigin/matrixone/pkg/util/json"
 	"github.com/matrixorigin/matrixone/pkg/vm/process"
 	"go.starlark.net/starlark"
+	"go.starlark.net/syntax"
 	"go.uber.org/zap"
 )
 
@@ -91,9 +92,18 @@ func (op *opBuiltInStarlark) buildFn(proc *process.Process, program string) erro
 		"jq": starlark.NewBuiltin("jq", moJq),
 	}
 
-	// execute the starlark program, ExecFile is deprecated but still widely used
-	// in starlark-go repo.
-	op.globals, err = starlark.ExecFile(op.th, "", program, op.predeclared)
+	op.globals, err = starlark.ExecFileOptions(
+		&syntax.FileOptions{
+			While:           true,
+			TopLevelControl: true,
+			GlobalReassign:  true,
+			Recursion:       true,
+		},
+		op.th,
+		"",
+		program,
+		op.predeclared,
+	)
 	if err != nil {
 		return err
 	}