Re: Added schema level support for publication. - Mailing list pgsql-hackers

From vignesh C
Subject Re: Added schema level support for publication.
Date
Msg-id CALDaNm36zN4OWKHLsgR86hoNjz8NkqXkFOnsohiAcn1cd9grMA@mail.gmail.com
Whole thread Raw
In response to RE: Added schema level support for publication.  ("houzj.fnst@fujitsu.com" <houzj.fnst@fujitsu.com>)
List pgsql-hackers
On Mon, Sep 6, 2021 at 6:56 AM houzj.fnst@fujitsu.com
<houzj.fnst@fujitsu.com> wrote:
>
> From Thur, Sep 2, 2021 2:33 PM vignesh C <vignesh21@gmail.com> wrote:
> > On Wed, Sep 1, 2021 at 6:58 AM houzj.fnst@fujitsu.com <houzj.fnst@fujitsu.com> wrote:
> > >
> > > Here are some other comments for v23-000x patches.
> > > 3)
> > >
> > > +                                                         .description =
> > "PUBLICATION SCHEMA",
> > > +                                                         .section =
> > SECTION_POST_DATA,
> > > +                                                         .createStmt
> > > + = query->data));
> > >
> > > Is it better to use something like 'PUBLICATION TABLES IN SCHEMA' to
> > > describe the schema level table publication ? Because there could be
> > > some other type publication such as 'ALL SEQUENCES IN SCHEMA' in the
> > > future, it will be better to make it clear that we only publish table in schema in
> > this patch.
> >
> > Modified
>
> Thanks for updating the patch.
>
> I think we might also need to mention the publication object 'table' in the
> following types:
>
> 1)
> +       /* OCLASS_PUBLICATION_SCHEMA */
> +       {
> +               "publication schema", OBJECT_PUBLICATION_SCHEMA
> +       },
>
> 2)
> +       PUBLICATIONOBJ_SCHEMA,          /* Schema type */
> +       PUBLICATIONOBJ_UNKNOWN          /* Unknown type */
> +} PublicationObjSpecType;
>
> 3)
> +       DO_PUBLICATION_SCHEMA,
>
> I think it might be to change the typename like XX_REL_IN_SCHEMA,
> and adjust the comments.

Thanks for the comments, this is handled in the v26 patch attached at [1]
[1] - https://www.postgresql.org/message-id/CALDaNm3EwAVma8n4YpV1%2BQWiccuVPxpqNfbbrUU3s3XTHcTXew%40mail.gmail.com

Regards,
Vignesh



pgsql-hackers by date:

Previous
From: vignesh C
Date:
Subject: Re: Added schema level support for publication.
Next
From: torikoshia
Date:
Subject: Re: RFC: Logging plan of the running query