avantgardnerio commented on code in PR #17347: URL: https://github.com/apache/datafusion/pull/17347#discussion_r2308160226
########## datafusion/common/src/config.rs: ########## @@ -727,6 +727,10 @@ config_namespace! { /// during aggregations, if possible pub enable_topk_aggregation: bool, default = true + /// When set to true, the optimizer will attempt to push limit operations + /// past window functions, if possible + pub enable_window_limits: bool, default = true Review Comment: I think it will come in handy for sqllogictesting (comparing results & plans), and as an "escape hatch" for a few versions of DataFusion so users can disable if they find a bug. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org