li-boxuan opened a new pull request, #10322: URL: https://github.com/apache/incubator-gluten/pull/10322
`RemoveSort` rule assumes if a Sort operator's parent doesn't require ordering, then it can be removed. This isn't always true, since in Spark there's no "required physical property" concept as in Cascades framework. [51768](https://github.com/apache/spark/pull/51768) is one example. Although I tend to believe it's a Spark bug, it also reveals the fact that it's dangerous to hold this assumption in Gluten. -- 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]
