You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by jo...@apache.org on 2019/06/10 23:10:38 UTC

[incubator-superset] branch master updated: Revert "Allow bigint to be used as time column in Presto" #7670 (#7678)

This is an automated email from the ASF dual-hosted git repository.

johnbodley pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 49390b3  Revert "Allow bigint to be used as time column in Presto" #7670 (#7678)
49390b3 is described below

commit 49390b323215aceac9c7ff57077120a7d13d693b
Author: John Bodley <45...@users.noreply.github.com>
AuthorDate: Mon Jun 10 16:10:20 2019 -0700

    Revert "Allow bigint to be used as time column in Presto" #7670 (#7678)
---
 superset/db_engine_specs/presto.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/superset/db_engine_specs/presto.py b/superset/db_engine_specs/presto.py
index 79c58b9..b29b003 100644
--- a/superset/db_engine_specs/presto.py
+++ b/superset/db_engine_specs/presto.py
@@ -355,8 +355,6 @@ class PrestoEngineSpec(BaseEngineSpec):
             return "from_iso8601_date('{}')".format(dttm.isoformat()[:10])
         if tt == 'TIMESTAMP':
             return "from_iso8601_timestamp('{}')".format(dttm.isoformat())
-        if tt == 'BIGINT':
-            return "to_unixtime(from_iso8601_timestamp('{}'))".format(dttm.isoformat())
         return "'{}'".format(dttm.strftime('%Y-%m-%d %H:%M:%S'))
 
     @classmethod