Remove obsolete FORCE option from REINDEX.

FORCE option has been marked "obsolete" since very old version 7.4
but existed for backwards compatibility. Per discussion on pgsql-hackers,
we concluded that it's no longer worth keeping supporting the option.

diff --git a/doc/src/sgml/ref/reindex.sgml b/doc/src/sgml/ref/reindex.sgml
index 0a4c7d45cbf8cd397cad38b8d5d7800cac59f776..998340c5db2b86f9dc83c15d7883e6eab92f2937 100644 (file)
--- a/doc/src/sgml/ref/reindex.sgml
+++ b/doc/src/sgml/ref/reindex.sgml
@@ -21,7 +21,7 @@ PostgreSQL documentation
 
  
 
-REINDEX { INDEX | TABLE | SCHEMA | DATABASE | SYSTEM } name [ FORCE ]
+REINDEX { INDEX | TABLE | SCHEMA | DATABASE | SYSTEM } name
 
  
 
@@ -150,15 +150,6 @@ REINDEX { INDEX | TABLE | SCHEMA | DATABASE | SYSTEM } 
      
     
    
-
-   
-    FORCE
-    
-     
-      This is an obsolete option; it is ignored if specified.
-     
-    
-   
   
  
 

diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index 88ec83c7c3884a4a3885d2c1a2125bb00ed35d2a..5818858a295e6a2be52f162883e020b464cfb578 100644 (file)
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -283,7 +283,7 @@ static Node *makeRecursiveViewSelect(char *relname, List *aliases, Node *query);
 
 %type    opt_lock lock_type cast_context
 %type    vacuum_option_list vacuum_option_elem
-%type     opt_force opt_or_replace
+%type     opt_or_replace
                opt_grant_grant_option opt_grant_admin_option
                opt_nowait opt_if_exists opt_with_data
 %type    opt_nowait_or_skip
@@ -7301,13 +7301,11 @@ opt_if_exists: IF_P EXISTS                      { $$ = TRUE; }
  *
  *     QUERY:
  *
- *     REINDEX type  [FORCE]
- *
- * FORCE no longer does anything, but we accept it for backwards compatibility
+ *     REINDEX type 
  *****************************************************************************/
 
 ReindexStmt:
-           REINDEX INDEX qualified_name opt_force
+           REINDEX INDEX qualified_name
                {
                    ReindexStmt *n = makeNode(ReindexStmt);
                    n->kind = REINDEX_OBJECT_INDEX;
@@ -7315,7 +7313,7 @@ ReindexStmt:
                    n->name = NULL;
                    $$ = (Node *)n;
                }
-           | REINDEX TABLE qualified_name opt_force
+           | REINDEX TABLE qualified_name
                {
                    ReindexStmt *n = makeNode(ReindexStmt);
                    n->kind = REINDEX_OBJECT_TABLE;
@@ -7323,7 +7321,7 @@ ReindexStmt:
                    n->name = NULL;
                    $$ = (Node *)n;
                }
-           | REINDEX SCHEMA name opt_force
+           | REINDEX SCHEMA name
                {
                    ReindexStmt *n = makeNode(ReindexStmt);
                    n->kind = REINDEX_OBJECT_SCHEMA;
@@ -7331,7 +7329,7 @@ ReindexStmt:
                    n->relation = NULL;
                    $$ = (Node *)n;
                }
-           | REINDEX SYSTEM_P name opt_force
+           | REINDEX SYSTEM_P name
                {
                    ReindexStmt *n = makeNode(ReindexStmt);
                    n->kind = REINDEX_OBJECT_SYSTEM;
@@ -7339,7 +7337,7 @@ ReindexStmt:
                    n->relation = NULL;
                    $$ = (Node *)n;
                }
-           | REINDEX DATABASE name opt_force
+           | REINDEX DATABASE name
                {
                    ReindexStmt *n = makeNode(ReindexStmt);
                    n->kind = REINDEX_OBJECT_DATABASE;
@@ -7349,10 +7347,6 @@ ReindexStmt:
                }
        ;
 
-opt_force: FORCE                                   {  $$ = TRUE; }
-           | /* EMPTY */                           {  $$ = FALSE; }
-       ;
-
 
 /*****************************************************************************
  *