haohuaijin commented on code in PR #21527:
URL: https://github.com/apache/datafusion/pull/21527#discussion_r3077968788


##########
datafusion/core/tests/physical_optimizer/filter_pushdown.rs:
##########
@@ -5146,3 +5150,350 @@ async fn 
test_hashjoin_dynamic_filter_pushdown_left_semi_join() {
     "
     );
 }
+
+// ==== Filter pushdown through SortExec tests ====
+
+/// FilterExec above a plain SortExec (no fetch) should be pushed below it.
+/// The scan supports pushdown, so the filter lands in the DataSourceExec.
+#[test]

Review Comment:
   due to 
   - https://github.com/apache/datafusion/issues/15886 
   the subquery's order by is removed by our sql parser layer, so i can't 
reproduce use sql



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