You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/10/27 06:21:02 UTC

[GitHub] [superset] john-bodley commented on pull request #17239: fix: revert "feat(sqla): apply time grain to all temporal groupbys"

john-bodley commented on pull request #17239:
URL: https://github.com/apache/superset/pull/17239#issuecomment-952578940


   @villebro I agree with your comment. Additionally a poor UX, i.e., something unintended and sub-optimal, can become the de facto norm based on user behavior. The issue then is future fixes—with good intend—are actually breaking changes given the saved formed data and thus a migration of some form (preferably non-breaking and thus not associated with the Superset 2.0 release) is likely needed.


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