builtins: put check_consistency on DistsqlBlocklist

Fixes cockroachdb#88222. Release note: None
ajwerner · Sep 26, 2022 · 77091fe · 77091fe
1 parent bbcb10f
commit 77091fe
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/pkg/sql/logictest/testdata/logic_test/builtin_function_notenant b/pkg/sql/logictest/testdata/logic_test/builtin_function_notenant
@@ -59,7 +59,7 @@ true
 
 # Fill a table with consistency check results. This used to panic.
 # See: https://github.com/cockroachdb/cockroach/issues/88222
-statement error pgcode XX000 no consistency checker configured
+statement ok
 CREATE TABLE conscheckresult AS (SELECT * FROM crdb_internal.check_consistency(false, '', ''));
 
 # Test crdb_internal commands which execute as root, but

diff --git a/pkg/sql/planner.go b/pkg/sql/planner.go
@@ -481,6 +481,7 @@ func internalExtendedEvalCtx(
 			SQLStatsController:             sqlStatsController,
 			SchemaTelemetryController:      schemaTelemetryController,
 			IndexUsageStatsController:      indexUsageStatsController,
+			ConsistencyChecker:             execCfg.ConsistencyChecker,
 			StmtDiagnosticsRequestInserter: execCfg.StmtDiagnosticsRecorder.InsertRequest,
 			RangeStatsFetcher:              execCfg.RangeStatsFetcher,
 		},

diff --git a/pkg/sql/sem/builtins/generator_builtins.go b/pkg/sql/sem/builtins/generator_builtins.go
@@ -394,8 +394,9 @@ var generators = map[string]builtinDefinition{
 
 	"crdb_internal.check_consistency": makeBuiltin(
 		tree.FunctionProperties{
-			Class:    tree.GeneratorClass,
-			Category: builtinconstants.CategorySystemInfo,
+			Class:            tree.GeneratorClass,
+			Category:         builtinconstants.CategorySystemInfo,
+			DistsqlBlocklist: true, // see #88222
 		},
 		makeGeneratorOverload(
 			tree.ArgTypes{