On Mon, Nov 10, 2025 at 4:22 PM vignesh C <[email protected]> wrote: > > On Mon, 10 Nov 2025 at 14:34, Shlok Kyal <[email protected]> wrote: > > > > While working on another thread, I found that in HEAD gram.y has > > grammar which was committed as part of this thread: > > ``` > > | CREATE PUBLICATION name FOR pub_obj_type_list opt_definition > > { > > CreatePublicationStmt *n = > > makeNode(CreatePublicationStmt); > > > > n->pubname = $3; > > n->pubobjects = (List *) $5; > > preprocess_pub_all_objtype_list($5, &n->for_all_tables, > > &n->for_all_sequences, > > yyscanner); > > n->options = $6; > > $$ = (Node *) n; > > } > > ``` > > > > Here we are assigning "n->pubobjects = (List *) $5". But later in the > > code this is not used anywhere for ALL TABLES/ ALL SEQUENCES > > publication. It is used for other publications (not ALL TABLES/ > > SEQUENCES) inside function "ObjectsInPublicationToOids" > > > > So are we required to assign "n->pubobjects" here? > > > > I have created a patch to remove this assignment. It passed "make > > check-world". > > I agree that this is not required for ALL TABLES/ALL SEQUENCES cases. >
I also agree. BTW, can we change pub_obj_type_list to pub_all_obj_type_list to be more explicit about this variant? If you agree then we can make similar changes (pub_obj_type->pub_all_obj_type) at the following places as well: + * CREATE PUBLICATION FOR ALL pub_obj_type [, ...] [WITH options] + * + * pub_obj_type is one of: + * -- With Regards, Amit Kapila.
