[GitHub] [superset] nytai commented on pull request #12903: fix(explore): hide time section for datasets with no time column

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

nytai commented on pull request #12903:
URL: https://github.com/apache/superset/pull/12903#issuecomment-773704683


   I largely agree with @ktmud's sentiments. I never quite understood why superset doesn't treat time just like any other calculated dimension or filter. I suspect it's due to time series datasets always being quite large and there's a high probably of users hosing the client. It could also be due to superset's early focus on time series data. It's a +1 from me for making superset a more general purpose data visualization tool with dimensions, metrics and filters as the core building blocks. 
   
   However, change management is important and this is a significant change that's likely to cause existing users to think it's a bug. We should find a way to gently transition to the ideal end state. I think having good docs and an FAQ section could be sufficient. Adding dismissable tooltips to explore to communicate the recent changes could also work. 


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