peter-toth commented on PR #8817: URL: https://github.com/apache/arrow-datafusion/pull/8817#issuecomment-1908029297
I kept my eye on this PR and IMO it looks good (my concers have been addressed) and actually a nice refactor that unifies many different derived trees into `PlanContext` and `ExprContext`. Also, thanks for the general multi-stage example @ozankabak. Once this PR gets merged I would rebase my https://github.com/apache/arrow-datafusion/pull/8664 to see if it can improve some of the transformations while also simplify the tansform logic. -- 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]
