cloud-fan commented on pull request #29021:
URL: https://github.com/apache/spark/pull/29021#issuecomment-654694004


   ```
   SMJ
        Sort
          Shuffle
        Sort
          HashAggregate
            Shuffle
   ```
   
   This is an interesting use case. We must be careful when dealing with it. 
The key of skew join handling is to split the skew partition into smaller 
parts. For `HashAggregate`, I'm not sure if this works, as now the values of 
the same key may exist in different after-split partitions. This makes 
`HashAggregate` incorrect.


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