Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit a6964bc

Browse files
committed
Remove unnecessary grammar symbols
Instead of publication_name_list, we can use name_list. We already refer to publications everywhere else by the 'name' or 'name_list' symbols, so this only improves consistency. Reviewed-by: https://www.postgresql.org/message-id/flat/3e3ccddb-41bd-ecd8-29fe-195e34d9886f%40enterprisedb.com Discussion: Tom Lane <tgl@sss.pgh.pa.us>
1 parent 8ae4ef4 commit a6964bc

File tree

1 file changed

+2
-18
lines changed

1 file changed

+2
-18
lines changed

src/backend/parser/gram.y

+2-18
Original file line numberDiff line numberDiff line change
@@ -414,15 +414,13 @@ static Node *makeRecursiveViewSelect(char *relname, List *aliases, Node *query);
414414
relation_expr_list dostmt_opt_list
415415
transform_element_list transform_type_list
416416
TriggerTransitions TriggerReferencing
417-
publication_name_list
418417
vacuum_relation_list opt_vacuum_relation_list
419418
drop_option_list
420419

421420
%type <list> group_by_list
422421
%type <node> group_by_item empty_grouping_set rollup_clause cube_clause
423422
%type <node> grouping_sets_clause
424423
%type <node> opt_publication_for_tables publication_for_tables
425-
%type <value> publication_name_item
426424

427425
%type <list> opt_fdw_options fdw_options
428426
%type <defelt> fdw_option
@@ -9512,7 +9510,7 @@ AlterPublicationStmt:
95129510
*****************************************************************************/
95139511

95149512
CreateSubscriptionStmt:
9515-
CREATE SUBSCRIPTION name CONNECTION Sconst PUBLICATION publication_name_list opt_definition
9513+
CREATE SUBSCRIPTION name CONNECTION Sconst PUBLICATION name_list opt_definition
95169514
{
95179515
CreateSubscriptionStmt *n =
95189516
makeNode(CreateSubscriptionStmt);
@@ -9524,20 +9522,6 @@ CreateSubscriptionStmt:
95249522
}
95259523
;
95269524

9527-
publication_name_list:
9528-
publication_name_item
9529-
{
9530-
$$ = list_make1($1);
9531-
}
9532-
| publication_name_list ',' publication_name_item
9533-
{
9534-
$$ = lappend($1, $3);
9535-
}
9536-
;
9537-
9538-
publication_name_item:
9539-
ColLabel { $$ = makeString($1); };
9540-
95419525
/*****************************************************************************
95429526
*
95439527
* ALTER SUBSCRIPTION name ...
@@ -9572,7 +9556,7 @@ AlterSubscriptionStmt:
95729556
n->options = $6;
95739557
$$ = (Node *)n;
95749558
}
9575-
| ALTER SUBSCRIPTION name SET PUBLICATION publication_name_list opt_definition
9559+
| ALTER SUBSCRIPTION name SET PUBLICATION name_list opt_definition
95769560
{
95779561
AlterSubscriptionStmt *n =
95789562
makeNode(AlterSubscriptionStmt);

0 commit comments

Comments
 (0)