superset-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-superset] bkyryliuk commented on pull request #9752: fix(mssql): CAST with AT produces statements with syntax errors
Date Thu, 07 May 2020 16:48:51 GMT

bkyryliuk commented on pull request #9752:
URL: https://github.com/apache/incubator-superset/pull/9752#issuecomment-625370820


   yeah we've looked into alternatives quite some time ago and sqlparse was the only feasible
option at a time. There are DB specific parsers, but they often implemented in different languages
e.g. java + antlr for hive and presto as I remember and using them as was mentioned will be
opening a can of warms. Explain query can provide some useful information as well, however
I am not sure about MS SQL specifics there. I'm curious what is a sqllachemy solution if it
supports query construction for MSSQL


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


Mime
View raw message