alamb commented on code in PR #21460:
URL: https://github.com/apache/datafusion/pull/21460#discussion_r3051263030


##########
datafusion/physical-plan/src/filter.rs:
##########
@@ -2572,4 +2576,67 @@ mod tests {
         );
         Ok(())
     }
+
+    /// Regression test: ProjectionExec on top of a FilterExec that already has

Review Comment:
   is it possble to trigger this issue from SQL? As in can this type of plan be 
constructed via SQL or does it need to be programatically created?
   
   I am thinking how we can test this at a higher level (this unit test is fine 
but I think it is so narrowly focused that wI worry we may be missing some 
other issues)
   
   For example if not via SQL, can we create a plan with LogicalPlanBuilder? 



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