From: | "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com> |
---|---|
To: | vignesh C <vignesh21(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | RE: [PATCH]Comment improvement in publication.sql |
Date: | 2021-08-03 03:06:15 |
Message-ID: | OS0PR01MB6113C7BB4F0466632396C7D7FBF09@OS0PR01MB6113.jpnprd01.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Monday, August 2, 2021 11:56 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
>
> Few minor suggestions:
> 1) Should we change below to "fail - tables can't be added, dropped or
> set to "FOR ALL TABLES" publications"
> --- fail - can't add to for all tables publication
> +-- fail - tables can't be added to or dropped from FOR ALL TABLES publications
Thanks for your kindly comments.
I'm not sure should we ignore that 'drop' should be followed by 'from', but I
think there's no misunderstandings. So, modified as you described.
Besides, in other file comments (src/test/subscription/t/100_bugs.pl) I saw FOR ALL TABLES without using quotes.
So I don't think we need the quotes, too.
> 2) Should we change this to "--fail - already existing publication"
> --- fail - already added
> +-- fail - already exists
Modified.
A modified patch is attached.
Regards
Tang
Attachment | Content-Type | Size |
---|---|---|
v2-improvement-on-comment.patch | application/octet-stream | 3.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Smith | 2021-08-03 03:22:24 | Re: row filtering for logical replication |
Previous Message | Andres Freund | 2021-08-03 02:56:55 | Re: A micro-optimisation for ProcSendSignal() |