villebro commented on issue #10468:
URL: 
https://github.com/apache/incubator-superset/issues/10468#issuecomment-666135550


   @cyang52 what I'm proposing is leaving the time fields there. If the user 
doesn't add the timestamp column to either `entity` or `groupby`, it will not 
be included in the query. However, if it is, then it will affect the chart. If 
we do make this change, we will need to migrate the chart metadata for existing 
boxplot charts to add the timestamp field to the new `entity` field to make it 
work like before.


----------------------------------------------------------------
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to