spark-test commented on PR #59889:
URL: https://github.com/apache/spark/pull/59889#issuecomment-6512923948
Thanks for the update. I checked 0f7bd96: the stale double-failure test is
gone, and `decimal remainder bounds exact retries by divisor scale` now pins
the divisor-scale term of the retry estimate. With `+ divisor.scale()` dropped
or negated, `Decimal(1L) % Decimal("3E-100")` would get an estimate of 1 or -99
instead of 101. It would then take the exact retry and return `1E-100`, which
fails the intercept. The test-coverage paragraph of the description also no
longer claims double-failure coverage.
<!-- SPARK_DEV_REVIEW_REPLY
{"feedback_id":"conversation:6485181587","thread_id":"conversation:6485181587","verdict_sha256":"5737e1cbc49d5ce6b3a83277363f0590f1695a9140f1942f900025fa121cabac"}
-->
--
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]
