We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TestFuzz
go1.20.1
Since the changes made in #12399, we saw TestFuzz was randomly stalling and timing out in the CI and locally.
Upgrade to go1.20.1 and run TestFuzz.
main, release-16.0
n/a
No response
The text was updated successfully, but these errors were encountered:
This issue will be fixed by #12399 thanks to 92d842f.
The reason why TestFuzz was timing out is because the test calls New
New
p := New(minSize, maxSize)
Where minSize is a random number between 0 and 16384. However, New uses minSize the following way:
minSize
0
16384
const multiplier = 2 ... curSize := minSize for curSize < maxSize { pools = append(pools, newSizedPool(curSize)) curSize *= multiplier }
Meaning that curSize *= multiplier will always be evaluated to 0 *= 2, and thus the loop condition curSize < maxSize will always be false.
curSize *= multiplier
0 *= 2
curSize < maxSize
false
Sorry, something went wrong.
frouioui
Successfully merging a pull request may close this issue.
Overview of the Issue
Since the changes made in #12399, we saw
TestFuzz
was randomly stalling and timing out in the CI and locally.Reproduction Steps
Upgrade to
go1.20.1
and runTestFuzz
.Binary Version
Operating System and Environment details
Log Fragments
No response
The text was updated successfully, but these errors were encountered: