Create a test case similar to SqlParserTest.testSqlOptions... the rest should follow...
On Tue, Jul 28, 2015 at 6:07 AM, Jacques Nadeau <[email protected]> wrote: > Ps, the word at the beginning of that email should have been "if" > On Jul 28, 2015 6:06 AM, "Jacques Nadeau" <[email protected]> wrote: > >> 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 >> >>
