ozankabak opened a new pull request, #13984:
URL: https://github.com/apache/datafusion/pull/13984

   ## Which issue does this PR close?
   
   None.
   
   ## Rationale for this change
   
   The function `swap_inputs` was missing for the SMJ operator. This is a 
continuation of #13910 that fills this gap.
   
   ## What changes are included in this PR?
   
   This PR implements the `swap_inputs` method for the SMJ operator, which 
basically is the same with its sisters for other operators (e.g. Hash Join). It 
also makes the `swap_join_according_to_unboundedness` utility function in 
`join_selection.rs` crate public to enable implementation of (and 
experimentation with) custom join rules by downstream projects.
   
   ## Are these changes tested?
   
   There are no explicit unit tests, but the sisters methods don't either. 
These are tested indirectly via optimization rules. We can consider whether to 
add such tests for all such methods in a future PR.
   
   ## Are there any user-facing changes?
   
   None.
   


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to