diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2013-06-05 13:32:53 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2013-06-05 13:33:33 -0400 |
| commit | 53eb835578536bcbb0c02ef1e1d5051ca8029285 (patch) | |
| tree | fcf57bab4d2ec0783e1b31a8c6a24fd170c78f76 | |
| parent | c28bfb35ef6ce9e9d82da00008554d0fc9c4a7fc (diff) | |
Put analyze_keyword back in explain_option_name production.
In commit 2c92edad48796119c83d7dbe6c33425d1924626d, 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.
| -rw-r--r-- | src/backend/parser/gram.y | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y index a8ff1d1ce46..3df7c046445 100644 --- a/src/backend/parser/gram.y +++ b/src/backend/parser/gram.y @@ -6980,6 +6980,7 @@ explain_option_elem: explain_option_name: NonReservedWord { $$ = $1; } + | analyze_keyword { $$ = "analyze"; } ; explain_option_arg: |