Put analyze_keyword back in explain_option_name production.

tglsfdc · tglsfdc · commit 940a85e5e1c3 · 2013-06-05T13:33:18.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
@@ -7920,6 +7920,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
`@@ -7920,6 +7920,7 @@ explain_option_elem:`
`7920`	`7920`
`7921`	`7921`	`explain_option_name:`
`7922`	`7922`	`NonReservedWord { $$ = $1; }`
	`7923`	`+ \| analyze_keyword { $$ = "analyze"; }`
`7923`	`7924`	`;`
`7924`	`7925`
`7925`	`7926`	`explain_option_arg:`