calcite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jacques Nadeau <jacq...@apache.org>
Subject Re: Support to reset an option
Date Tue, 28 Jul 2015 13:06:29 GMT
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" <skatkam@maprtech.com> 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

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message