prakharjain09 opened a new pull request #29318:
URL: https://github.com/apache/spark/pull/29318


   ### What changes were proposed in this pull request?
   This PR fixes issues relate to Canonicalization of FileSourceScanExec when 
it contains unused DPP Filter.
   
   ### Why are the changes needed?
   
   As part of PlanDynamicPruningFilter rule, the unused DPP Filter are simply 
replaced by `DynamicPruningExpression(TrueLiteral)` so that they can be 
avoided. But these unnecessary`DynamicPruningExpression(TrueLiteral)` partition 
filter inside the FileSourceScanExec affects the canonicalization of the node 
and so in many cases, this can prevent ReuseExchange from happening.
   
   This PR fixes this issue by ignoring the unused DPP filter in the `def 
doCanonicalize` method. 
   
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   Added UT.


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

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