increase in SUBBUCKETS has performance impact
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
golang-1.14 (Ubuntu) |
New
|
Medium
|
Unassigned | ||
Focal |
New
|
Medium
|
Unassigned | ||
Groovy |
Won't Fix
|
Medium
|
Unassigned | ||
Hirsute |
Won't Fix
|
Medium
|
Unassigned | ||
golang-1.15 (Ubuntu) |
New
|
Medium
|
Unassigned | ||
Focal |
Invalid
|
Undecided
|
Unassigned | ||
Groovy |
Invalid
|
Undecided
|
Unassigned | ||
Hirsute |
Won't Fix
|
Medium
|
Unassigned |
Bug Description
[impact]
a negative performance impact is found with the increase in SUBBUCKETS
[test case]
TBD
[regression potential]
any regression would likely cause performance issues
[scope]
this is needed only for golang 1.14 and 1.15; the patch is not added in 1.13 or earlier, nor in 1.16.
Specifically, this is needed in f/g/h in the specific golang versions as shown:
$ grep -E 'SUBBUCKETS += 32' */*/golang-
golang-
golang-
golang-
golang-
[other info]
this was added to golang-1.14 with the patch:
debian/
and golong-1.15 with the similar patch:
debian/
There does not appear to be any documentation or explanation for why the patch modifies SUBBUCKETS.
Upstream, this appears to have been done with the commit 15eaa870e14195c
Changed in golang-1.15 (Ubuntu Focal): | |
status: | New → Invalid |
Changed in golang-1.15 (Ubuntu Groovy): | |
status: | New → Invalid |
Changed in golang-1.15 (Ubuntu Hirsute): | |
importance: | Undecided → Medium |
Changed in golang-1.15 (Ubuntu Groovy): | |
importance: | Undecided → Medium |
Changed in golang-1.15 (Ubuntu Focal): | |
importance: | Undecided → Medium |
importance: | Medium → Undecided |
Changed in golang-1.15 (Ubuntu Groovy): | |
importance: | Medium → Undecided |
Changed in golang-1.14 (Ubuntu Focal): | |
importance: | Undecided → Medium |
Changed in golang-1.14 (Ubuntu Groovy): | |
importance: | Undecided → Medium |
Changed in golang-1.14 (Ubuntu Hirsute): | |
importance: | Undecided → Medium |
The Groovy Gorilla has reached end of life, so this bug will not be fixed for that release