AmoghAtreya commented on PR #43176: URL: https://github.com/apache/superset/pull/43176#issuecomment-5704538238
Thanks @gabotorresruiz — the hoist is out of the shared serializer. `set_query_context_form_data` no longer invents a range from filters or `cache_values`, so chart-data / async `get_time_filter()` matches master again. Tabular callers pass the original range into `execute_tabular_query`; `QueryObject.time_range` stays unset. I skipped `>=` / `<` reconstruction because of the `get_table` comparison-filter edge. Added the chart-shaped `NO_TIME_RANGE` case, the `get_table time_column` case, moved the hoist tests onto `execute_tabular_query`, and pinned `TZ=UTC` on the rollover test. Will add update the PR description too. -- 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]
