michellethomas opened a new pull request #4590: Removing escape_sql so we dont double escape URL: https://github.com/apache/incubator-superset/pull/4590 @john-bodley Removing escape_sql because this seems to be handled by `qry.compile(engine)` [here](https://github.com/apache/incubator-superset/blob/master/superset/connectors/sqla/models.py#L411) and both of them escaping double escapes.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services