[GitHub] [superset] prosdev0107 commented on pull request #19010: fix(explore-sqllab): make that Timestamp column keep the Is temporal …

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

prosdev0107 commented on pull request #19010:
URL: https://github.com/apache/superset/pull/19010#issuecomment-1059416280


   > 
   
   
   
   > It might be use to add repro steps to the PR description for historical purposes, and for anyone who wants to review/test. In this case:
   > 
   > 1. Navigate to the SQL Lab.
   > 2. Run a SELECT statement including a timestamp column.
   > 3. Click on **EXPLORE**.
   > 4. Create a new dataset.
   > 5. Note that the timestamp column is available to be selected on the time column drop-down.
   > 6. Navigate back to the SQL Lab.
   > 7. Run the same query again.
   > 8. Click on **EXPLORE**.
   > 9. Choose to overwrite the dataset you have just created.
   > 10. Note that the timestamp column is not available to be used on the time column drop-down.
   > 
   > Also, is it possible to add a test for this?
   
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

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