Re: Support EXCEPT for ALL SEQUENCES publications - Mailing list pgsql-hackers

From Peter Smith
Subject Re: Support EXCEPT for ALL SEQUENCES publications
Date
Msg-id CAHut+PuC7qCdBUcCL1DeQEuRJsp-PU-n7_Jd4Hs-REn9M5ZO4g@mail.gmail.com
Whole thread
Responses Re: Support EXCEPT for ALL SEQUENCES publications
List pgsql-hackers
Only trivial comments for v16*

//////
v16-0001
//////

======
src/bin/pg_dump/pg_dump.c

getPublictions:

1.
  /*
- * Get the list of tables for publications specified in the EXCEPT
- * TABLE clause.
+ * Get the list of tables and sequences for publications specified in
+ * the EXCEPT clause.
  *

Perhaps this comment should say that EXCEPT (TABLE ...) is introduced
in PG19, and EXCEPT (SEQUENCE ...) is introduced in PG20. That way
there are no surprises about the different version checks that follow.

======
src/test/subscription/t/037_except.pl

2.
+# Check the initial data on subscriber
+$result = $node_subscriber->safe_psql('postgres',
+ "SELECT last_value, is_called FROM seq_excluded_in_pub1");
+is($result, '1|f', 'sequences in EXCEPT list is excluded');

typo: plural. Should be "sequences in the EXCEPT list are excluded"

//////
v16-0002
//////

======
src/test/subscription/t/037_except.pl

1.
+is($result, '1|f', 'sequences in EXCEPT list is excluded');

typo: plural. Should be "sequences in the EXCEPT list are excluded"

======
Kind Regards,
Peter Smith.
Fujitsu Australia



pgsql-hackers by date:

Previous
From: Nico Williams
Date:
Subject: Re: Global temporary tables
Next
From: Chao Li
Date:
Subject: Re: Clear base backup progress reporting on error