ivoson opened a new pull request, #53652:
URL: https://github.com/apache/spark/pull/53652

   ### What changes were proposed in this pull request?
   Enable checksum based indeterminate shuffle retry by default.
   
   Increase jvm memory size to 6g for `sql` module tests, as test case 
[SPARK-48037: Fix SortShuffleWriter lacks shuffle write related metrics 
resulting in potentially inaccurate 
data](https://github.com/apache/spark/blob/316322cbcb55ff5c1b4e479bc2aae12babdae534/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala#L2696)
 set shuffle partition as `16777216` which will need more memory for computing 
order independent shuffle checksum.
   
   ### Why are the changes needed?
   As checksum based solution is more accurate to detect indeterminate shuffle 
output changes, propose to enable it by default to avoid query correctness 
issues caused by indeterminate shuffle retry.
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   Existing UTs.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No


-- 
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: [email protected]

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