Put analyze_keyword back in explain_option_name production.

tglsfdc · tglsfdc · commit 48b5120977e2 · 2013-06-05T13:33:05.000-04:00
In commit 2c92eda, I broke "EXPLAIN (ANALYZE)" syntax, because I mistakenly thought that ANALYZE/ANALYSE were only partially reserved and thus would be included in NonReservedWord; but actually they're fully reserved so they still need to be called out here. A nicer solution would be to demote these words to type_func_name_keyword status (they can't be less than that because of "VACUUM [ANALYZE] ColId"). While that works fine so far as the core grammar is concerned, it breaks ECPG's grammar for reasons I don't have time to isolate at the moment. So do this for the time being. Per report from Kevin Grittner. Back-patch to 9.0, like the previous commit.
diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
@@ -8316,6 +8316,7 @@ explain_option_elem:
 
 explain_option_name:
 			NonReservedWord			{ $$ = $1; }
+			| analyze_keyword		{ $$ = "analyze"; }
 		;
 
 explain_option_arg:

Original file line number	Diff line number	Diff line change
`@@ -8316,6 +8316,7 @@ explain_option_elem:`
`8316`	`8316`
`8317`	`8317`	`explain_option_name:`
`8318`	`8318`	`NonReservedWord { $$ = $1; }`
	`8319`	`+ \| analyze_keyword { $$ = "analyze"; }`
`8319`	`8320`	`;`
`8320`	`8321`
`8321`	`8322`	`explain_option_arg:`