[GitHub] [arrow-datafusion] liukun4515 commented on pull request #7247: Change benefits_from_partitioning flag to vector

["liukun4515 (via GitHub)" <gi...@apache.org>]

liukun4515 commented on PR #7247:
URL: https://github.com/apache/arrow-datafusion/pull/7247#issuecomment-1674864494

   it's reasonable for me about the changes.
   ```
   hash join: collect left
       left
       repartition
         right
   ```
   
   The changes from remind me a issue: 
   When the partition of hash  is `collect left`, the right side can be repartitioned or not be repartitioned. 
   How to measure which method is better.
   Multi partition can use multi-thread resource but has the side effect of a new operator of `repartitionexec` after this changes
   
   cc @mingmwang 
   
   
   


-- 
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-unsubscribe@arrow.apache.org

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