duongcongtoai commented on PR #16186: URL: https://github.com/apache/datafusion/pull/16186#issuecomment-2917689696
So here are my thoughts (this plan is to split the work in smaller PRs) while avoid breaking things as much as possible: 1. we introduce 3 optimizors, declared in the order below: - DependentJoinRewriter - OldOptimizors (`DecorrelateLateralJoin`, `ScalarSubqueryToJoin`, `DecorrelatePredicateSubquery`) - NewOptimizor 2. We add a if condition inside DependentJoinRewriter, that if the total subquery depth is 1, it doesn't change the query plan at all => This allow the old optimizors to work as expected 3. Develop the newOptimizor -- 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
