morrySnow commented on code in PR #11673: URL: https://github.com/apache/doris/pull/11673#discussion_r943166742
########## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/physical/PhysicalQuickSort.java: ########## @@ -30,56 +30,30 @@ import org.apache.commons.lang3.StringUtils; import java.util.List; -import java.util.Objects; import java.util.Optional; /** - * Physical sort plan. + * Physical quick sort plan. */ -public class PhysicalHeapSort<CHILD_TYPE extends Plan> extends PhysicalUnary<CHILD_TYPE> { +public class PhysicalQuickSort<CHILD_TYPE extends Plan> extends AbstractPhysicalSort<CHILD_TYPE> { Review Comment: without limit and offset, be use quick sort to do sort indeed -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org