Re: adding partitioned tables to publications

From: Amit Langote <amitlangote09(at)gmail(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Petr Jelinek <petr(at)2ndquadrant(dot)com>, Rafia Sabih <rafia(dot)pghackers(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: adding partitioned tables to publications
Date: 2020-04-07 06:44:50
Message-ID: CA+HiwqE2-6U=XzsBZ9efzPatFDgrgGybBEj7vm-kUQL8DM4BWw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Apr 7, 2020 at 12:04 AM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
>
> On Mon, Apr 6, 2020 at 10:25 PM Peter Eisentraut
> <peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> > On 2020-04-03 16:25, Amit Langote wrote:
> > > On Fri, Apr 3, 2020 at 6:34 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > >> I am checking test coverage at the moment and should have the patches
> > >> ready by sometime later today.
> > >
> > > Attached updated patches.
> >
> > Committed 0001 now. I'll work on the rest tomorrow.
>
> Thank you. I have rebased the one remaining.

I updated the patch to make the following changes:

* Rewrote the tests to match in style with those committed yesterday
* Renamed all variables that had pubasroot in it to have pubviaroot
instead to match the publication parameter
* Updated pg_publication catalog documentation

--
Thank you,

Amit Langote
EnterpriseDB: http://www.enterprisedb.com

Attachment Content-Type Size
v18-0001-Allow-publishing-partition-changes-via-ancestors.patch application/octet-stream 59.8 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2020-04-07 06:47:44 Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions
Previous Message Tatsuro Yamada 2020-04-07 06:40:34 Re: Feature improvement: can we add queryId for pg_catalog.pg_stat_activity view?