Reminiscent commented on code in PR #18784: URL: https://github.com/apache/doris/pull/18784#discussion_r1194826748
########## fe/fe-core/src/main/java/org/apache/doris/planner/PartitionSortNode.java: ########## Review Comment: Yes, most of the parts in this file are copied from the `SortNode`. But I think some are used to generate the `SortInfo`which will be passed to BE side. By the way, I can't figure out all the function very precise. I'll remove some unrelated functions as much as possible. So feel free to point out which function maybe used in the `nereids`. -- 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]
