moved to use the query_runner.annotate_query flag so we won't get the…

… SQL comment
KiiCorp · Mar 17, 2014 · 642a2b6 · 642a2b6
1 parent 96d6f6f
commit 642a2b6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/data/query_runner_script.py b/data/query_runner_script.py
@@ -17,10 +17,6 @@ def query_runner(query):
             json_data = None
             error = None
 
-            # Remove the SQL comment that Redash adds
-            if query.find("/*") > -1 and query.find("*/") > -1:
-                query = query[query.find("*/")+3:]
-
             # Poor man's protection against running scripts from output the scripts directory
             if connection_string.find("../") > -1:
                 return None, "Scripts can only be run from the configured scripts directory"
@@ -48,4 +44,5 @@ def query_runner(query):
 
         return json_data, error
 
+    query_runner.annotate_query = False
     return query_runner
diff --git a/data/query_runner_url.py b/data/query_runner_url.py
@@ -15,10 +15,6 @@ def query_runner(query):
 
             query = query.strip()
 
-            # Remove the SQL comment that Redash adds
-            if query.find("/*") > -1 and query.find("*/") > -1:
-                query = query[query.find("*/")+3:]
-
             if base_url is not None and base_url != "":
                 if query.find("://") > -1:
                     return None, "Accepting only relative URLs to '%s'" % base_url
@@ -45,4 +41,5 @@ def query_runner(query):
 
         return json_data, error
 
+    query_runner.annotate_query = False
     return query_runner