[GitHub] [superset] tritonrc commented on issue #13009: Incorrect SELECT * statement previewing tables with timestamp partitions

[GitBox <gi...@apache.org>]

tritonrc commented on issue #13009:
URL: https://github.com/apache/superset/issues/13009#issuecomment-776857855


   Hi, I ran into this same issue yesterday and after digging around the code I landed on a fix.  In my case, the fields were coming back marked as `Time` type.  Looking at https://github.com/apache/superset/blob/master/superset/db_engine_specs/presto.py#L576
   
   I insert two lines to cover the `Time` type at line 579:
   
   ```python
   if tt == utils.TemporalType.TIME:
               return f"""from_iso8601_timestamp('{dttm.isoformat(timespec="microseconds")}')"""  # pylint: disable=line-too-long
   ``` 
   
   As part of digging into this, I ran across the recently landed PR https://github.com/apache/superset/pull/12861 which might help here as well.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org