Abdulrehman-PIAIC80387 commented on PR #42082: URL: https://github.com/apache/superset/pull/42082#issuecomment-5019501141
@rusackas You're right — ran a SQL diff across 9 scenarios (baseline, series_limit, groupby, TEMPORAL_RANGE filter, various is_timeseries/grain combos): **identical output in every case**. Fix is a no-op. Generated SQL for the reporter's scenario is `WHERE dttm >= '2025-05-01' AND dttm < '2025-05-28' GROUP BY month` — WHERE is correct, so the aggregated value should already exclude May 29-31. Closing this. @maranafaamen if you can share the actual broken SQL you saw, happy to reopen with a real fix. -- 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]
