Skip to content

Commit

Permalink
PG17 regress sanity: fix error unrecognized alter database option tab…
Browse files Browse the repository at this point in the history
…lespace seen in database vanilla test (#7764)

Disable DDL propagation for the vanilla test suite. This enables the
vanilla `database ` test to pass, where previously it was correctly
returning `ERROR: unrecognized ALTER DATABASE option: tablespace`
because release-13.0 does not propagate this ALTER DATABASE variant.

We (Citus team) discussed cherry picking
[#7253](#7253) from main to
release-13.0 because it does propagate ALTER DATABASE tablespace option
(as well as a couple of others) but decided fixing the regress test was
not the proper context for that. The fix disables
`citus.enable_metadata_sync` when running vanilla, we discussed
disabling `citus.enable_create_database_propagation` but this is not in
release-13.0.
  • Loading branch information
colm-mchugh authored Nov 25, 2024
1 parent 1d0111a commit d5f067a
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/test/regress/pg_regress_multi.pl
Original file line number Diff line number Diff line change
Expand Up @@ -511,6 +511,9 @@ sub generate_hba

# we disable some restrictions for local objects like local views to not break postgres vanilla test behaviour.
push(@pgOptions, "citus.enforce_object_restrictions_for_local_objects=false");

# we disable metadata synchronization to enable postgres vanilla DDL tests to pass.
push(@pgOptions, "citus.enable_metadata_sync=false");
}

if ($useMitmproxy)
Expand Down

0 comments on commit d5f067a

Please sign in to comment.