zhztheplayer opened a new pull request, #5409: URL: https://github.com/apache/incubator-gluten/pull/5409
This PR adds a new physical operator TopNTransformer in Velox backend mapping to Velox's `core::TopNNode`. By doing this, we can remove the hacky limit + sort coherent validation code, and the executed plan used by `TakeOrderedAndProjectExecTransformer` will correctly map to the actual physical plan in Velox task. This PR is a phase 1 because the actual Velox plan is still hidden by `TakeOrderedAndProjectExecTransformer` although limit+sort was replaced by top-n. We may eventually deprecate `TakeOrderedAndProjectExecTransformer` and expose more execution details to query planner. -- 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]
