NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit17d436d

committed

Remove obsolete FORCE option from REINDEX.

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

1 parent7320681 commit17d436dCopy full SHA for 17d436d

File tree

2 files changed

-23

lines changed

doc/src/sgml/ref
- reindex.sgml
src/backend/parser
- gram.y

2 files changed

-23

lines changed

`‎doc/src/sgml/ref/reindex.sgml`

Lines changed: 1 addition & 10 deletions

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ PostgreSQL documentation`
`21`	`21`
`22`	`22`	`<refsynopsisdiv>`
`23`	`23`	`<synopsis>`
`24`		`-REINDEX { INDEX \| TABLE \| SCHEMA \| DATABASE \| SYSTEM } <replaceable class="PARAMETER">name</replaceable> [ FORCE ]`
	`24`	`+REINDEX { INDEX \| TABLE \| SCHEMA \| DATABASE \| SYSTEM } <replaceable class="PARAMETER">name</replaceable>`
`25`	`25`	`</synopsis>`
`26`	`26`	`</refsynopsisdiv>`
`27`	`27`
`@@ -150,15 +150,6 @@ REINDEX { INDEX \| TABLE \| SCHEMA \| DATABASE \| SYSTEM } <replaceable class="PARAM`
`150`	`150`	`</para>`
`151`	`151`	`</listitem>`
`152`	`152`	`</varlistentry>`
`153`		`-`
`154`		`- <varlistentry>`
`155`		`- <term><literal>FORCE</literal></term>`
`156`		`- <listitem>`
`157`		`- <para>`
`158`		`- This is an obsolete option; it is ignored if specified.`
`159`		`- </para>`
`160`		`- </listitem>`
`161`		`- </varlistentry>`
`162`	`153`	`</variablelist>`
`163`	`154`	`</refsect1>`
`164`	`155`

`‎src/backend/parser/gram.y`

Lines changed: 7 additions & 13 deletions

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ static Node makeRecursiveViewSelect(char relname, List aliases, Node query);`
`283`	`283`
`284`	`284`	`%type<ival>opt_locklock_typecast_context`
`285`	`285`	`%type<ival>vacuum_option_listvacuum_option_elem`
`286`		`-%type<boolean>opt_forceopt_or_replace`
	`286`	`+%type<boolean>opt_or_replace`
`287`	`287`	`opt_grant_grant_optionopt_grant_admin_option`
`288`	`288`	`opt_nowaitopt_if_existsopt_with_data`
`289`	`289`	`%type<ival>opt_nowait_or_skip`
`@@ -7301,45 +7301,43 @@ opt_if_exists: IF_P EXISTS{ $$ = TRUE; }`
`7301`	`7301`	`*`
`7302`	`7302`	`*QUERY:`
`7303`	`7303`	`*`
`7304`		`- *REINDEX type <name> [FORCE]`
`7305`		`- *`
`7306`		`- * FORCE no longer does anything, but we accept it for backwards compatibility`
	`7304`	`+ *REINDEX type <name>`
`7307`	`7305`	`*****************************************************************************/`
`7308`	`7306`
`7309`	`7307`	`ReindexStmt:`
`7310`		`-REINDEXINDEXqualified_nameopt_force`
	`7308`	`+REINDEXINDEXqualified_name`
`7311`	`7309`	`{`
`7312`	`7310`	`ReindexStmt *n = makeNode(ReindexStmt);`
`7313`	`7311`	`n->kind = REINDEX_OBJECT_INDEX;`
`7314`	`7312`	`n->relation =$3;`
`7315`	`7313`	`n->name =NULL;`
`7316`	`7314`	`$$ = (Node *)n;`
`7317`	`7315`	`}`
`7318`		`-\|REINDEXTABLEqualified_nameopt_force`
	`7316`	`+\|REINDEXTABLEqualified_name`
`7319`	`7317`	`{`
`7320`	`7318`	`ReindexStmt *n = makeNode(ReindexStmt);`
`7321`	`7319`	`n->kind = REINDEX_OBJECT_TABLE;`
`7322`	`7320`	`n->relation =$3;`
`7323`	`7321`	`n->name =NULL;`
`7324`	`7322`	`$$ = (Node *)n;`
`7325`	`7323`	`}`
`7326`		`-\|REINDEXSCHEMAnameopt_force`
	`7324`	`+\|REINDEXSCHEMAname`
`7327`	`7325`	`{`
`7328`	`7326`	`ReindexStmt *n = makeNode(ReindexStmt);`
`7329`	`7327`	`n->kind = REINDEX_OBJECT_SCHEMA;`
`7330`	`7328`	`n->name =$3;`
`7331`	`7329`	`n->relation =NULL;`
`7332`	`7330`	`$$ = (Node *)n;`
`7333`	`7331`	`}`
`7334`		`-\|REINDEXSYSTEM_Pnameopt_force`
	`7332`	`+\|REINDEXSYSTEM_Pname`
`7335`	`7333`	`{`
`7336`	`7334`	`ReindexStmt *n = makeNode(ReindexStmt);`
`7337`	`7335`	`n->kind = REINDEX_OBJECT_SYSTEM;`
`7338`	`7336`	`n->name =$3;`
`7339`	`7337`	`n->relation =NULL;`
`7340`	`7338`	`$$ = (Node *)n;`
`7341`	`7339`	`}`
`7342`		`-\|REINDEXDATABASEnameopt_force`
	`7340`	`+\|REINDEXDATABASEname`
`7343`	`7341`	`{`
`7344`	`7342`	`ReindexStmt *n = makeNode(ReindexStmt);`
`7345`	`7343`	`n->kind = REINDEX_OBJECT_DATABASE;`
`@@ -7349,10 +7347,6 @@ ReindexStmt:`
`7349`	`7347`	`}`
`7350`	`7348`	`;`
`7351`	`7349`
`7352`		`-opt_force:FORCE{$$ =TRUE; }`
`7353`		`-\|/* EMPTY*/{$$ =FALSE; }`
`7354`		`-;`
`7355`		`-`
`7356`	`7350`
`7357`	`7351`	`/*****************************************************************************`
`7358`	`7352`	`*`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit17d436d

File tree

2 files changed

2 files changed

`‎doc/src/sgml/ref/reindex.sgml`

`‎src/backend/parser/gram.y`

0 commit comments