Steve Carlin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/21194
Change subject: IMPALA-12934: Added Calcite parsing files to Impala
......................................................................
IMPALA-12934: Added Calcite parsing files to Impala
Adding the framework to create our own parsing syntax for Impala using
the base Calcite Parser.jj file.
The Parser.jj file here was grabbed from Calcite 1.36. So with this commit,
we are using the same parsing analysis as Calcite 1.36. Any changes made
on top of the Parser.jj file or the config.fmpp file in the futre are Impala
specific changes, so a diff can be done from this commit to see all the Impala
parsing changes.
The config.fmpp file was grabbed from Calcite 1.36 default_config.fmpp. It does
have two very small modifications from the original Calcite fmpp file in order
to fix compilation issues within Impala
1) the entire json is wrapped with a "data {}" tag
2) the class used is ImpalaSqlParserImpl as opposed to the Calcite SqlParserImpl
class to prevent naming collisions with Calcite.
There's no unit test needed since there is no functional change. The Calcite
planner will eventually make changes in the ".jj" file to support the
differences
between the Impala parser and the Calcite parser.
Change-Id: If756b5ea8beb85661a30fb5d029e74ebb6719767
---
M java/calcite-planner/pom.xml
A
java/calcite-planner/src/main/java/org/apache/impala/calcite/parserimpl/codegen/config.fmpp
A
java/calcite-planner/src/main/java/org/apache/impala/calcite/parserimpl/codegen/templates/Parser.jj
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteQueryParser.java
M java/pom.xml
5 files changed, 9,627 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/21194/2
--
To view, visit http://gerrit.cloudera.org:8080/21194
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If756b5ea8beb85661a30fb5d029e74ebb6719767
Gerrit-Change-Number: 21194
Gerrit-PatchSet: 2
Gerrit-Owner: Steve Carlin <[email protected]>
