Re: Added schema level support for publication.

vignesh C <vignesh21@gmail.com>

From: vignesh C <vignesh21@gmail.com>

To: Amit Kapila <amit.kapila16@gmail.com>

Cc: Greg Nancarrow <gregn4422@gmail.com>, Masahiko Sawada <sawada.mshk@gmail.com>, "houzj.fnst@fujitsu.com" <houzj.fnst@fujitsu.com>, "tanghy.fnst@fujitsu.com" <tanghy.fnst@fujitsu.com>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, Tom Lane <tgl@sss.pgh.pa.us>, Peter Smith <smithpb2250@gmail.com>, Ajin Cherian <itsajin@gmail.com>, Rahila Syed <rahilasyed90@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>

Date: 2021-10-25T07:41:23Z

Lists: pgsql-hackers

Attachments

v47-0001-Allow-publishing-the-tables-of-schema.patch (text/x-patch) patch v47-0001
v47-0002-Add-tap-tests-for-the-schema-publication-feature.patch (text/x-patch) patch v47-0002
v47-0003-Add-new-pg_publication_objects-view-to-display-T.patch (text/x-patch) patch v47-0003

On Mon, Oct 25, 2021 at 10:52 AM Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> On Fri, Oct 22, 2021 at 8:56 PM vignesh C <vignesh21@gmail.com> wrote:
> >
>
> I am getting a compilation error in the latest patch on HEAD. I think
> was relying on some variable removed by a recent commit
> 92316a4582a5714d4e494aaf90360860e7fec37a. While looking at that
> compilation error, I observed that we don't need the second and third
> parameters in pg_dump.c/getPublicationNamespaces() as those are not
> getting used.

I have fixed this in the v47 version attached.

Regards,
Vignesh

Commits

Include schema/table publications even with exclude options in dump.
- 4aa6fa3cd0a2 18.0 landed
Rename some enums to use TABLE instead of REL.
- b3812d0b9bcf 15.0 landed
Add tap tests for the schema publications.
- 6b0f6f79eef2 15.0 landed
Allow publishing the tables of schema.
- 5a2832465fd8 15.0 landed
In pg_dump, use simplehash.h to look up dumpable objects by OID.
- 92316a4582a5 15.0 cited