Support parenthesized syntax for CLUSTER without a table name.

b5913f6120 added a parenthesized syntax for CLUSTER, but it
requires specifying a table name.  This is unlike commands such as
VACUUM and ANALYZE, which do not require specifying a table in the
parenthesized syntax.  This change resolves this inconsistency.
This is preparatory work for a follow-up commit that will move the
unparenthesized syntax to the "Compatibility" section of the
CLUSTER documentation.

Reviewed-by: Melanie Plageman, Michael Paquier
Discussion: https://postgr.es/m/CAAKRu_bc5uHieG1976kGqJKxyWtyQt9yvktjsVX%2Bi7NOigDjOA%40mail.gmail.com

diff --git a/doc/src/sgml/ref/cluster.sgml b/doc/src/sgml/ref/cluster.sgml
index e308e2ce912ec14acf126c006f8bff82a5ea0250..b6d5655c194359fa7317d171c15186f729975fe0 100644 (file)
--- a/doc/src/sgml/ref/cluster.sgml
+++ b/doc/src/sgml/ref/cluster.sgml
@@ -21,7 +21,7 @@ PostgreSQL documentation
 
  
 
-CLUSTER [ ( option [, ...] ) ] table_name [ USING index_name ]
+CLUSTER [ ( option [, ...] ) ] [ table_name [ USING index_name ] ]
 CLUSTER [ VERBOSE ] [ table_name [ USING index_name ] ]
 
 where option can be one of:

diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index 91793cb2effb00fc2050e5f5829fcf1fa4385f88..7a44a374e4918126947f0312de3e7cc608023313 100644 (file)
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -11553,7 +11553,7 @@ CreateConversionStmt:
 /*****************************************************************************
  *
  *     QUERY:
- *             CLUSTER (options)  [ USING  ]
+ *             CLUSTER (options) [  [ USING  ] ]
  *             CLUSTER [VERBOSE] [  [ USING  ] ]
  *             CLUSTER [VERBOSE]  ON  (for pre-8.3)
  *
@@ -11569,6 +11569,15 @@ ClusterStmt:
                    n->params = $3;
                    $$ = (Node *) n;
                }
+           | CLUSTER '(' utility_option_list ')'
+               {
+                   ClusterStmt *n = makeNode(ClusterStmt);
+
+                   n->relation = NULL;
+                   n->indexname = NULL;
+                   n->params = $3;
+                   $$ = (Node *) n;
+               }
            /* unparenthesized VERBOSE kept for pre-14 compatibility */
            | CLUSTER opt_verbose qualified_name cluster_index_specification
                {
@@ -11581,6 +11590,7 @@ ClusterStmt:
                        n->params = lappend(n->params, makeDefElem("verbose", NULL, @2));
                    $$ = (Node *) n;
                }
+           /* unparenthesized VERBOSE kept for pre-17 compatibility */
            | CLUSTER opt_verbose
                {
                    ClusterStmt *n = makeNode(ClusterStmt);