sql: refactor InternalExecutor on EvalCtx

Minor refactor to remove duplicate InternalExecutor definition
on EvalCtx and in the tree package.

We already have 2 interface declarations for InternalExecutor
in sql and sqlutil, this refactor hopefully makes the dependencies
a little more clear.

Release note: None

pkg/ccl/streamingccl/streamingutils/BUILD.bazel

@@ -9,7 +9,9 @@ go_library(
         "//pkg/jobs",
         "//pkg/jobs/jobspb",
         "//pkg/kv",
+        "//pkg/security",
         "//pkg/sql/sem/tree",
+        "//pkg/sql/sessiondata",
         "//pkg/streaming",
         "//pkg/util/hlc",
         "//pkg/util/protoutil",

pkg/sql/faketreeeval/BUILD.bazel

@@ -6,6 +6,7 @@ go_library(
     importpath = "github.com/cockroachdb/cockroach/pkg/sql/faketreeeval",
     visibility = ["//visibility:public"],
     deps = [
+        "//pkg/kv",
         "//pkg/security",
         "//pkg/sql/parser",
         "//pkg/sql/pgwire/pgcode",

pkg/sql/row/BUILD.bazel

@@ -97,7 +97,6 @@ go_test(
         "//pkg/sql/rowenc",
         "//pkg/sql/rowinfra",
         "//pkg/sql/sem/tree",
-        "//pkg/sql/sqlutil",
         "//pkg/storage",
         "//pkg/testutils",
         "//pkg/testutils/serverutils",

pkg/sql/sem/builtins/BUILD.bazel

@@ -61,7 +61,6 @@ go_library(
         "//pkg/sql/sqlerrors",
         "//pkg/sql/sqlliveness",
         "//pkg/sql/sqltelemetry",
-        "//pkg/sql/sqlutil",
         "//pkg/sql/types",
         "//pkg/streaming",
         "//pkg/util",