public inbox for [email protected]
help / color / mirror / Atom feedFrom: Alvaro Herrera <[email protected]>
To: Rahila Syed <[email protected]>
Cc: Amit Kapila <[email protected]>
Cc: Tomas Vondra <[email protected]>
Cc: Peter Smith <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Subject: Re: Column Filtering in Logical Replication
Date: Fri, 17 Dec 2021 12:58:50 -0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAH2L28uU2yPSnRGcM-BU3HFKL6NS3LTeLjgACz6kvNBR6wQA1Q@mail.gmail.com>
On 2021-Dec-17, Rahila Syed wrote:
> > This means that we need to support changing the column list of a
> > table in a publication. I'm looking at implementing some form of
> > ALTER PUBLICATION for that.
>
> I think right now the patch contains support only for ALTER
> PUBLICATION.. ADD TABLE with column filters. In order to achieve
> changing the column lists of a published table, I think we can extend
> the ALTER TABLE ..SET TABLE syntax to support specification of column
> list.
Yeah, that's what I was thinking too.
> > So this whole thing can be reduced to just this:
>
> > if (att_map != NULL && !bms_is_member(att->attnum, att_map))
> > continue; /* that is, don't send this attribute */
>
> I agree the condition can be shortened now. The long if condition was
> included because initially the feature allowed specifying filters
> without replica identity columns(sent those columns internally without
> user having to specify).
Ah, true, I had forgotten that. Thanks.
> > 900 + * the table is partitioned. Run a recursive query to iterate through all
> > 901 + * the parents of the partition and retreive the record for the parent
> > 902 + * that exists in pg_publication_rel.
> > 903 + */
>
> The above comment in fetch_remote_table_info() can be changed as the
> recursive query is no longer used.
Oh, of course.
I'll finish some loose ends and submit a v10, but it's still not final.
--
Álvaro Herrera 39°49'30"S 73°17'W — https://www.EnterpriseDB.com/
"Right now the sectors on the hard disk run clockwise, but I heard a rumor that
you can squeeze 0.2% more throughput by running them counterclockwise.
It's worth the effort. Recommended." (Gerry Pourwelle)
view thread (167+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: Column Filtering in Logical Replication
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox