Of you're making changes there anyway,  can you make two additional changes:

Allow "alter session" to be optional for setting.
Allow a multipart identifier (so we don't have to quote a.b.c (same as
Schema or column identifiers in project lists).

This would substantially improve usability.
On Jul 27, 2015 6:53 PM, "Sudheesh Katkam" <[email protected]> wrote:

> Hello developers,
>
> To support these statements as part of Apache Drill's SQL parser extension:
>
> ALTER SESSION RESET `option name`
> ALTER SYSTEM RESET `option name`
> ALTER SESSION RESET ALL
> ALTER SYSTEM RESET ALL
>
> I added the required implementation files, including changes to the config
> file (keywords, statementParserMethods fields). However, I get a
> ParseException because the parser resolves to SetSqlOption statement, which
> I assume is due to a lookahead of 2. How do I go about resolving this?
>
> Thank you,
> Sudheesh

Reply via email to