Skip to content

Commit

Permalink
sqlsmith: disable partial index mutator
Browse files Browse the repository at this point in the history
This is a temporary fix for cockroachdb#55718.

Release justification: This fixes a flaky sqlsmith test which is
blocking builds on the release branch.

Release note: None
  • Loading branch information
mgartner committed Oct 20, 2020
1 parent 314007f commit b877286
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion pkg/internal/sqlsmith/setup.go
Original file line number Diff line number Diff line change
Expand Up @@ -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))
Expand Down

0 comments on commit b877286

Please sign in to comment.