zhztheplayer opened a new pull request, #11072: URL: https://github.com/apache/incubator-gluten/pull/11072
The PR's an improvement following #10763. It decouples `ColumnarToColumnarExec` from `ColumnarToColumnarTransition` to make them both can be solely implemented. If a class inherits from `ColumnarToColumnarExec` but not from `ColumnarToColumnarTransition` at the same time, it will not be considered as a transition, so it's not going to be removed by RemoveTransitions rule. After the refactor, `VeloxResizeBatchesExec` will not inherit `ColumnarToColumnarTransition` and it will have nothing to do with the transition planner. So developers don't have to consider the transition logic when working with it or other similar unary columnar operators. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
