Remove unecessary handling of %%

apache · May 4, 2017 · 2d66baa · 2d66baa
1 parent 59a6f44
commit 2d66baa
Showing 1 changed file with 0 additions and 21 deletions.
diff --git a/superset/connectors/sqla/models.py b/superset/connectors/sqla/models.py
@@ -413,27 +413,6 @@ def get_sqla_query(  # sqla
             metrics_exprs = []
 
         if granularity:
-            @compiles(ColumnClause)
-            def visit_column(element, compiler, **kw):
-                """Patch for sqlalchemy bug
-
-                TODO: sqlalchemy 1.2 release should be doing this on its own.
-                Patch only if the column clause is specific for DateTime
-                set and granularity is selected.
-                """
-                text = compiler.visit_column(element, **kw)
-                try:
-                    if (
-                            element.is_literal and
-                            hasattr(element.type, 'python_type') and
-                            type(element.type) is DateTime
-                    ):
-                        text = text.replace('%%', '%')
-                except NotImplementedError:
-                    # Some elements raise NotImplementedError for python_type
-                    pass
-                return text
-
             dttm_col = cols[granularity]
             time_grain = extras.get('time_grain_sqla')