From 032c80866f6dcc6bfc9441205bf6bd1ec0908b2f Mon Sep 17 00:00:00 2001 From: Marcus Gartner Date: Mon, 19 Oct 2020 19:14:56 -0700 Subject: [PATCH] sqlsmith: disable partial index mutator This is a temporary fix for #55718. Release justification: This fixes a flaky sqlsmith test which is blocking builds on the release branch. Release note: None --- pkg/internal/sqlsmith/setup.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/internal/sqlsmith/setup.go b/pkg/internal/sqlsmith/setup.go index 9d6ddb0a61e6..064d6ec954ee 100644 --- a/pkg/internal/sqlsmith/setup.go +++ b/pkg/internal/sqlsmith/setup.go @@ -70,8 +70,13 @@ func randTables(r *rand.Rand) string { stmts := rowenc.RandCreateTables(r, "table", r.Intn(5)+1, mutations.ForeignKeyMutator, mutations.StatisticsMutator, - mutations.PartialIndexMutator, + // TODO(mgartner): Re-enable the partial index mutator once it is aware + // that it should not mutate unique indexes on foreign key references. + // mutations.PartialIndexMutator, ) + // Satisfy the linter's desire to have mutations.PartialIndexMutator be + // used. + var _ = mutations.PartialIndexMutator for _, stmt := range stmts { sb.WriteString(tree.SerializeForDisplay(stmt))