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 2021/08/12 06:08:41 UTC

[GitHub] [spark] cloud-fan commented on pull request #33711: Fix bug in disable unnecessary bucketed scan

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


   The fix LGTM. If a table has only one bucket, it can satisfy `AllTuples` and no shuffle is needed for aggregate without group keys. However, if we disable bucket scan, then `AllTuples` is not satisfied any more and the query returns wrong result.
   
   @IonutBoicuAms can you open a JIRA ticket and also enrich the PR description a little bit to explain why the bug happens?


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