Open stoyanov-x opened 6 years ago
I don't expect Uber has enough interest in MS SQL support to implement it internally, but I'm sure they'd be happy to merge new dialects if provided (@h4v0kh3l1 can you confirm or deny? :D). FWIW, I'm happy to provide guidance for anyone taking this on (for MS SQL or any other dialect).
That is accurate. We don't have a use case for MS SQL at Uber, but it is certainly within the scope of this project to expand our supported dialects.
And likewise with @dlthomas, I can also provide guidance on doing so.
@dlthomas @h4v0kh3l1 Hi guys i would like to get some guidance on how can i implement Redshift/Spark SQL Dialect.
@etrabelsi As a general high-level approach, start with copying one of the dialect parsers similar to Redshift/Spark SQL, and modifying the language features as the respective SQL documentation says. The tests contain a wealth of statements that should or should not pass parsing, and you'll want to modify those to fit the dialect.
It'll be an action item on me to publish guidelines for developing new dialect parsers, and for now let's start with the following:
As the package uploader, we'll need to work out the publishing process once the dialect is ready to roll, but we can defer to when we get there.
@dlthomas LMK if I missed something.
Hopefully that's enough to get you started. If you have follow-ups or more specific questions, let's open a new issue and start a conversation there, so we can keep this issue on MS SQL.
Are there any plans for MS SQL support? I think there are quite a few companies which could benefit from this.