mikebridge commented on PR #44454: URL: https://github.com/apache/superset/pull/44454#issuecomment-6129287642
CodeAnt correctly identifies the selected-axis replacement in time-series offsets and separate-inner-bound group-limit queries. This predates the master merge: the three filter helpers are AST-identical before and after it, and master already discards temporal-range filters when generated bounds are emitted. Existing regression tests explicitly pin replacement of same-axis ranges; this PR additionally preserves independent ranges on other date columns and distinct same-axis ranges in the main query. The documentation was too broad, so I clarified those comparison/group-limit exceptions without changing query behavior. All 9 focused temporal-filter tests and documentation hooks pass. Implemented with Codex and independently reviewed by Claude. -- 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]
