giftig commented on issue #30052: URL: https://github.com/apache/superset/issues/30052#issuecomment-3146348223
I had a look at the PR; it looks like it's _technically_ solved this by coercing it into a number of seconds, but it's not really added interval support, just a workaround. If there's currently no interval support on the frontend for other engines, it might need some thought of how to consistently represent an interval (it should ideally display as something similar to `28 days 16:30:00` like you can see in my screenshot, but that's just the string format postgres provides and we'd want to represent and then format it in a cross-engine way) Currently you'd be using an interval but just see `2476800` in that field, and it's not at all clear that represents just over a day (or even if it's a number of seconds, milliseconds, days...) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
