xzj7019 commented on code in PR #26993:
URL: https://github.com/apache/doris/pull/26993#discussion_r1393643488


##########
fe/fe-core/src/test/java/org/apache/doris/nereids/rules/rewrite/EliminateSortTest.java:
##########
@@ -26,7 +26,7 @@
 /**
  * column prune ut.
  */
-public class EliminateSortTest extends TestWithFeService implements 
MemoPatternMatchSupported {
+class EliminateSortTest extends TestWithFeService implements 
MemoPatternMatchSupported {

Review Comment:
   better to add ut to cover "select * from (select xxx order by xxx) a left 
join (select xxx ) b on a.id =b.id limit n;" also.



-- 
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]

Reply via email to