tanelk commented on pull request #30093:
URL: https://github.com/apache/spark/pull/30093#issuecomment-713222032
> With the proposed logic, the initial `global` value will be false, which
will lead to the `Repartition` case to return `false`, and the
`recursiveRemoveSort` will fail to eliminate the child local sort `Sort('b.asc,
false, _)`
>
> Can you provide an example when your logic can eliminate local child sorts
inside a local sort while the current logic cannot?
Ah yes, It seems we wouln't have to change the `canEliminateSort` at all.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
