genzgd commented on issue #28014: URL: https://github.com/apache/superset/issues/28014#issuecomment-2053586091
I think the basic problem is that toStartOfDay only accepts Date/DateTime arguments. The toDateTime intermediate step is required to not break String arguments in particular. I don't see a way to preserve that critical functionality while handling the timezone aware use case you are struggling with. I'm open to creative ideas here, but I don't know how to make your 'UTC' case work here while not breaking use cases that actually use the local and/or server timezone. -- 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-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
