Closed
Bug 869100
Opened 12 years ago
Closed 12 years ago
complete token bucket a/b test
Categories
(Core :: Networking: HTTP, defect)
Tracking
()
RESOLVED
FIXED
mozilla23
People
(Reporter: mcmanus, Assigned: mcmanus)
References
Details
Attachments
(1 file)
16.01 KB,
patch
|
mayhemer
:
review+
|
Details | Diff | Splinter Review |
This patch removes the A/B testing for the Token Bucket implementation.
With one exception, the data didn't really show any differences. The one exception was a configuration that relied only on reduced parallelism and a very slow token bucket - that profile was slower in the data than the others. So we can say that the token bucket doesn't regress things. I think it doesn't show an improvement, where it clearly does in targetted local testing, simply because the enhancement only applies to a certain edge cast which is not yet very common. I expect it to become more common in the future as data sizes increase and IW=10 becomes more common.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #745979 -
Flags: review?(honzab.moz)
![]() |
||
Updated•12 years ago
|
Attachment #745979 -
Flags: review?(honzab.moz) → review+
Assignee | ||
Comment 2•12 years ago
|
||
Comment 3•12 years ago
|
||
Assignee: nobody → mcmanus
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in
before you can comment on or make changes to this bug.
Description
•