You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/02/17 23:45:02 UTC

[GitHub] [spark] sigmod commented on pull request #35552: [SPARK-38237][SQL][SS] Rename back StatefulOpClusteredDistribution to HashClusteredDistribution

sigmod commented on pull request #35552:
URL: https://github.com/apache/spark/pull/35552#issuecomment-1043634678


   > As it was removed and renamed before, do we have any place that needs to use HashClusteredDistribution now?
   >  HashPartitioning with subset of grouping keys satisfying ClusteredDistribution, can be a tradeoff. 
   
   Since ClusteredDistribution is not strictly better than HashClusteredDistribution for all queries and vice versa, I think we should at least have a config for sending HashClusteredDistribution down as a distribution requirement, so that users can toggle the config for their queries when using ClusteredDistribution is worse than HashClusteredDistribution. 
   


-- 
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: reviews-unsubscribe@spark.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org