Skip to content

Commit cbafb53

Browse files
committed
sql: deprecate SCRUB command
Trimmed the `SCRUB` implementation down to a notice directing users to the `INSPECT` command. The `pkg/sql/scrub` errors package is left intact as it's still used in a number of other packages. A test helper is updated to use `INSPECT` and `SHOW INSPECT ERRORS` commands. Fixes: #148274 Epic: CRDB-55075 Release note (sql change): The `EXPERIMENTAL SCRUB` command is deprecated. The `INSPECT` command should be used for data consistency validation.
1 parent 78edad0 commit cbafb53

File tree

26 files changed

+53
-2949
lines changed

26 files changed

+53
-2949
lines changed

pkg/BUILD.bazel

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2310,7 +2310,6 @@ GO_TARGETS = [
23102310
"//pkg/sql/schemachanger:schemachanger",
23112311
"//pkg/sql/schemachanger:schemachanger_test",
23122312
"//pkg/sql/schemaobjectlimit:schemaobjectlimit",
2313-
"//pkg/sql/scrub/scrubtestutils:scrubtestutils",
23142313
"//pkg/sql/scrub:scrub",
23152314
"//pkg/sql/sem/asof:asof",
23162315
"//pkg/sql/sem/asof:asof_test",

pkg/ccl/partitionccl/BUILD.bazel

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ go_test(
3737
"drop_test.go",
3838
"main_test.go",
3939
"partition_test.go",
40-
"scrub_test.go",
4140
"zone_test.go",
4241
],
4342
embed = [":partitionccl"],
@@ -69,9 +68,6 @@ go_test(
6968
"//pkg/sql/lexbase",
7069
"//pkg/sql/parser",
7170
"//pkg/sql/randgen",
72-
"//pkg/sql/rowenc",
73-
"//pkg/sql/scrub",
74-
"//pkg/sql/scrub/scrubtestutils",
7571
"//pkg/sql/sem/tree",
7672
"//pkg/sql/tests",
7773
"//pkg/sql/types",

0 commit comments

Comments
 (0)