You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2018/06/22 23:52:55 UTC
[GitHub] conglei opened a new issue #5272: LIMIT statement is not correctly
caught by current regular expression
conglei opened a new issue #5272: LIMIT statement is not correctly caught by current regular expression
URL: https://github.com/apache/incubator-superset/issues/5272
The current logic of catching `LIMIT` statement has an assumption that it will be always at the very end of a sql query. Due to this, the table content preview is broken on sqlite, since `LIMIT 100 OFFSET 0` will be attached (from SqlAlchemy library). It results an extra `LIMIT 1000000` being added and cause the fatal error.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org