Horton
Menu

Re: Column Filtering in Logical Replication

Alvaro Herrera <alvherre@alvh.no-ip.org>

View thread
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: vignesh C <vignesh21@gmail.com>, Rahila Syed <rahilasyed90@gmail.com>, Tomas Vondra <tomas.vondra@enterprisedb.com>, Peter Smith <smithpb2250@gmail.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2021-09-24T13:25:09Z
Lists: pgsql-hackers 
On 2021-Sep-23, Amit Kapila wrote:

> Alvaro, do you have any thoughts on these proposed grammar changes?

Yeah, I think pubobj_name remains a problem in that you don't know its
return type -- could be a String or a RangeVar, and the user of that
production can't distinguish.  So you're still (unnecessarily, IMV)
stashing an object of undetermined type into ->object.

I think you should get rid of both pubobj_name and pubobj_expr and do
somethine like this:

/* FOR TABLE and FOR ALL TABLES IN SCHEMA specifications */
PublicationObjSpec:	TABLE ColId
					{
						$$ = makeNode(PublicationObjSpec);
						$$->pubobjtype = PUBLICATIONOBJ_TABLE;
						$$->rangevar = makeRangeVarFromQualifiedName($1, NULL, @1, yyscanner);
					}
			| TABLE ColId indirection
					{
						$$ = makeNode(PublicationObjSpec);
						$$->pubobjtype = PUBLICATIONOBJ_TABLE;
						$$->rangevar = makeRangeVarFromQualifiedName($1, $2, @1, yyscanner);
					}
			| ALL TABLES IN_P SCHEMA ColId
					{
						$$ = makeNode(PublicationObjSpec);
						$$->pubobjtype = PUBLICATIONOBJ_REL_IN_SCHEMA;
						$$->name = $4;
					}
			| ALL TABLES IN_P SCHEMA CURRENT_SCHEMA	/* XXX should this be "IN_P CURRENT_SCHEMA"? */
					{
						$$ = makeNode(PublicationObjSpec);
						$$->pubobjtype = PUBLICATIONOBJ_CURRSCHEMA;
						$$->name = $4;
					}
			| ColId
					{
						$$ = makeNode(PublicationObjSpec);
						$$->name = $1;
						$$->pubobjtype = PUBLICATIONOBJ_CONTINUATION;
					}
			| ColId indirection
					{
						$$ = makeNode(PublicationObjSpec);
						$$->rangevar = makeRangeVarFromQualifiedName($1, $2, @1, yyscanner);
						$$->pubobjtype = PUBLICATIONOBJ_CONTINUATION;
					}
			| CURRENT_SCHEMA
					{
						$$ = makeNode(PublicationObjSpec);
						$$->pubobjtype = PUBLICATIONOBJ_CURRSCHEMA;
					}
		;

so in AlterPublicationStmt you would have stanzas like

			| ALTER PUBLICATION name ADD_P pub_obj_list
				{
					AlterPublicationStmt *n = makeNode(AlterPublicationStmt);
					n->pubname = $3;
					n->pubobjects = preprocess_pubobj_list($5);
					n->action = DEFELEM_ADD;
					$$ = (Node *)n;
				}

where preprocess_pubobj_list (defined right after processCASbits and
somewhat mimicking it and SplitColQualList) takes all
PUBLICATIONOBJ_CONTINUATION and turns them into either
PUBLICATIONOBJ_TABLE entries or PUBLICATIONOBJ_REL_IN_SCHEMA entries,
depending on what the previous entry was.  (And of course if there is no
previous entry, raise an error immediately).  Note that node
PublicationObjSpec now has two fields, one for RangeVar and another for
a plain name, and tables always use the second one, except when they are
continuations, but of course those continuations that use name are
turned into rangevars in the preprocess step.  I think that would make
the code in ObjectsInPublicationToOids less messy.

(I don't think using the string "CURRENT_SCHEMA" is a great solution.
Did you try having a schema named CURRENT_SCHEMA?)

I verified that bison is happy with the grammar I proposed; I also
verified that you can add opt_column_list to the stanzas for tables, and
it remains happy.

-- 
Álvaro Herrera              Valdivia, Chile  —  https://www.EnterpriseDB.com/
Y una voz del caos me habló y me dijo
"Sonríe y sé feliz, podría ser peor".
Y sonreí. Y fui feliz.
Y fue peor.

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Doc: Explain about Column List feature.

  2. Doc: fix column list vs. replica identity rules.

  3. Prohibit combining publications with different column lists.

  4. Fix the check to limit sync workers.

  5. Wait for subscription to sync in t/031_column_list.sql

  6. Move prattrs to the pg_publication_rel section in docs

  7. Allow specifying column lists for logical replication

  8. Fix row filters with multiple publications

  9. Fix publish_as_relid with multiple publications

  10. Add some additional tests for row filters in logical replication.

  11. Add index on pg_publication_rel.prpubid

  12. Avoid using DefElemAction in AlterPublicationStmt

  13. Small cleanups related to PUBLICATION framework code

  14. Add PublicationTable and PublicationRelInfo structs

  15. Fix various concurrency issues in logical replication worker launching