New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 8 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Sep 2010
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug
M-7

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

Run A/B experiments on connection per host limits.

Project Member Reported by willchan@chromium.org, May 18 2010

Issue description

Currently we're at 6 connections per host.  Need to run more experiments.  
See  issue 12066  for some history.
 
Labels: Mstone-6
Status: Assigned
Ziad is running these experiments for now.  Ziad: punt this back to me if you don't finish them.
Labels: OS-All
Cleaning up mstone:6 bugs, default assumption is that bugs w/ no os are os-all
Labels: -Mstone-6 Mstone-7

Comment 5 by bugdro...@gmail.com, Jul 27 2010

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=53832 

------------------------------------------------------------------------
r53832 | ziadh@chromium.org | 2010-07-27 13:13:51 -0700 (Tue, 27 Jul 2010) | 7 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_main.cc?r1=53832&r2=53831

We would like to refine the numbers we started off with in the first experiment (then set to [4, 6, 8, 16] connections -- see http://codereview.chromium.org/2658006). Overall, 16 connections did not compare favorably to either 6 or 8 connections. At this point we'd like to modify and narrow down our range to span 5 to 9 connections.


r=jar
BUG= 44491 

Review URL: http://codereview.chromium.org/3077001
------------------------------------------------------------------------

Comment 6 by bugdro...@gmail.com, Jul 28 2010

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=54000 

------------------------------------------------------------------------
r54000 | ziadh@chromium.org | 2010-07-28 12:50:18 -0700 (Wed, 28 Jul 2010) | 10 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/472/src/chrome/browser/browser_main.cc?r1=54000&r2=53999

Merge 53832 - We would like to refine the numbers we started off with in the first experiment (then set to [4, 6, 8, 16] connections -- see http://codereview.chromium.org/2658006). Overall, 16 connections did not compare favorably to either 6 or 8 connections. At this point we'd like to modify and narrow down our range to span 5 to 9 connections.


r=jar
BUG= 44491 

Review URL: http://codereview.chromium.org/3077001

TBR=ziadh@chromium.org
Review URL: http://codereview.chromium.org/3017038
------------------------------------------------------------------------

Comment 7 by jar@chromium.org, Sep 20 2010

Status: Fixed
The experiments were run, and the uber-summary is that:

"6 is actually a decent value to be using. 7 connections seemed to perform slightly better in some circumstances, but probably not enough to make us want to switch to it."

We probably need to rip-out this experiment if we're fully satisfied with the results, or wait until we have yet better Page Load metrics, and revalidate results before tearing out it out.
is there a way to test this

Comment 9 by jar@chromium.org, Oct 18 2010

Statistics gathering don't really impact correctness, so it shouldn't be a test issue.

You could look at gathered stats, and verify that we're getting *some* stats for each category of test, and to do this we'd need the list of histograms that are plausible.

In general, the programmer does this checking (and gets value out of the test, or does not).  I think you can mark this as not-a-test-issue..
Spoke with Ziad.  no test needed
Project Member

Comment 11 by bugdroid1@chromium.org, Oct 12 2012

Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 12 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Area-Internals -Internals-Network -Mstone-7 M-7 Cr-Internals Cr-Internals-Network
Project Member

Comment 13 by bugdroid1@chromium.org, Mar 13 2013

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Sign in to add a comment