-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Spark & Presto Sql #1
Comments
I am also interested in spark sql, presto sql too @melin why there is only one g4 file for presto: https://github.com/prestodb/presto/blob/master/presto-parser/src/main/antlr4/com/facebook/presto/sql/parser/SqlBase.g4 ? |
Lexical and grammar combined in one file |
I would love to add Spark and Presto to this package, but I won't have time to do so over the next while. If you'd like to submit a pull request with the changes, that would be the fastest way to get this implemented. You can follow the docs on building and testing new grammars here. |
Spark Antlr4 Files:
https://github.com/apache/spark/tree/master/sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser
Presto Antlr4 file:
https://github.com/prestodb/presto/blob/master/presto-parser/src/main/antlr4/com/facebook/presto/sql/parser/SqlBase.g4
The text was updated successfully, but these errors were encountered: