irenjj commented on PR #17110: URL: https://github.com/apache/datafusion/pull/17110#issuecomment-3193472284
> Could you share how to merge this safely into mainbranch? should we maintain some feature flag, and let 2 framework of subquery decorrelation exists at the same time (current scattered optimizor enabled by default, living along side with our new optimizor, which will be added to CI for bug observation + more feature supported) Maybe we can add a param(such as `enable_general_decorrelate`) to enable decorrelate in this new rule. I think there is a long way to go to safely use this new general decorrelation framework. -- 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
