zhztheplayer opened a new pull request, #5575: URL: https://github.com/apache/incubator-gluten/pull/5575
The patch improves EnumerabledApplier to move RewriteSparkPlanRulesManager from heuristic rule list to RAS rule list. 1. Add guard [canOffload](https://github.com/apache/incubator-gluten/compare/main...zhztheplayer:wip-rewrite?expand=1#diff-1910d744e31ed430ae74eeef6518879e0a9a61728a34283ce42fa6b1372a7397R201-R231) to check if a vanilla aggregate is eligible to offload actually offloading it. This can be considered a follow-up to https://github.com/apache/incubator-gluten/pull/4628 since we don't actually support offloading of aggregate that has is pure-vanilla and has not been handled by post-project pulling routine. 2. Rename API `TransformSingleNode` to `OffloadSingleNode`. 3. Minor refactor on `RewriteSparkPlanRulesManager` by adding API `RewriteSingleNode`. 4. In EnumerableApplier, remove `RewriteSparkPlanRulesManager`, and add relevant rewrite rules to RAS rule list. -- 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]
