From e1af421f0c7426391e7124d1bae5aae9f8a49792 Mon Sep 17 00:00:00 2001 From: Michelle Thomas Date: Fri, 9 Mar 2018 15:17:41 -0800 Subject: [PATCH] Removing escape_sql so we dont double escape --- superset/connectors/sqla/models.py | 2 -- superset/db_engine_specs.py | 9 --------- 2 files changed, 11 deletions(-) diff --git a/superset/connectors/sqla/models.py b/superset/connectors/sqla/models.py index 3b636e8fa287..ef8c68ebb33e 100644 --- a/superset/connectors/sqla/models.py +++ b/superset/connectors/sqla/models.py @@ -432,8 +432,6 @@ def get_from_clause(self, template_processor=None, db_engine_spec=None): from_sql = self.sql if template_processor: from_sql = template_processor.process_template(from_sql) - if db_engine_spec: - from_sql = db_engine_spec.escape_sql(from_sql) from_sql = sqlparse.format(from_sql, strip_comments=True) return TextAsFrom(sa.text(from_sql), []).alias('expr_qry') return self.get_sqla_table() diff --git a/superset/db_engine_specs.py b/superset/db_engine_specs.py index 73f867729352..e026ccfb397c 100644 --- a/superset/db_engine_specs.py +++ b/superset/db_engine_specs.py @@ -145,11 +145,6 @@ def _allowed_file(filename): BaseEngineSpec.df_to_db(**df_to_db_kwargs) - @classmethod - def escape_sql(cls, sql): - """Escapes the raw SQL""" - return sql - @classmethod def convert_dttm(cls, target_type, dttm): return "'{}'".format(dttm.strftime('%Y-%m-%d %H:%M:%S')) @@ -552,10 +547,6 @@ def adjust_database_uri(cls, uri, selected_schema=None): uri.database = database return uri - @classmethod - def escape_sql(cls, sql): - return re.sub(r'%%|%', '%%', sql) - @classmethod def convert_dttm(cls, target_type, dttm): tt = target_type.upper()