| From: | "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Amit Langote <amitlangote09(at)gmail(dot)com> |
| Subject: | RE: pg_get_publication_tables() output duplicate relid |
| Date: | 2021-11-29 09:07:41 |
| Message-ID: | OS0PR01MB57164559C4C7F7F27943979494669@OS0PR01MB5716.jpnprd01.prod.outlook.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Nov 24, 2021 4:48 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
> On Mon, Nov 22, 2021 at 12:55 PM Amit Langote <amitlangote09(at)gmail(dot)com>
> wrote:
> >
> > On Fri, Nov 19, 2021 at 2:28 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
> wrote:
> > > On Fri, Nov 19, 2021 at 7:19 AM Amit Langote <amitlangote09(at)gmail(dot)com>
> wrote:
> > > > As in,
> > > > do we know of any replication (initial/streaming) misbehavior
> > > > caused by the duplicate partition OIDs in this case or is the only
> > > > problem that pg_publication_tables output looks odd?
> > >
> > > The latter one but I think either we should document this or change
> > > it as we can't assume users will follow what subscriber-side code does.
> >
> > On second thought, I agree that de-duplicating partitions from this
> > view is an improvement.
> >
>
> Fair enough. Hou-San, Can you please submit the updated patch after fixing
> any pending comments including the test case?
Attach the updated patch which address the comments so far.
The patch only adds a testcase in publication.sql because the duplicate
output doesn't cause unexpected behavior in pub-sub test.
Best regards,
Hou zj
| Attachment | Content-Type | Size |
|---|---|---|
| v2-0001-de-duplicate-the-result-of-pg_get_publication_tables.patch | application/octet-stream | 2.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | kuroda.hayato@fujitsu.com | 2021-11-29 09:57:22 | RE: [Proposal] Add foreign-server health checks infrastructure |
| Previous Message | Ken Kato | 2021-11-29 09:05:52 | [PATCH] DROP tab completion |