berkaysynnada commented on PR #8891: URL: https://github.com/apache/arrow-datafusion/pull/8891#issuecomment-1896136045
> API users can explicitly call `transform_up()` and then `transform_down()` (2 pass) if this is needed. Do you have a real use case for this? Since we set out to simplify and provide easy to use TreeNode API and its related implementations as much as possible, IMO we need to reach a simple state at the first step. As far as I observed, there is no use for this version of `transform()`, and it mixes the `VisitRecursion` logic with transform logic (early return option is a newly introduced feature for `transform()` methods). It may make sense to add such things when they are needed. -- 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]
