Commit 9d93e0c4 authored by ckrasic's avatar ckrasic Committed by Commit bot

QUIC - add UMA histogram to track retransmit rate for large uploads.

R=asvitkine@chromium.org, rch@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2342663003
Cr-Commit-Position: refs/heads/master@{#418719}
parent 194e7e00
...@@ -371,6 +371,13 @@ QuicChromiumClientSession::~QuicChromiumClientSession() { ...@@ -371,6 +371,13 @@ QuicChromiumClientSession::~QuicChromiumClientSession() {
UMA_HISTOGRAM_COUNTS("Net.QuicSession.MtuProbesSent", UMA_HISTOGRAM_COUNTS("Net.QuicSession.MtuProbesSent",
connection()->mtu_probe_count()); connection()->mtu_probe_count());
if (stats.packets_sent >= 100) {
// Used to monitor for regressions that effect large uploads.
UMA_HISTOGRAM_COUNTS_1000(
"Net.QuicSession.PacketRetransmitsPerMille",
1000 * stats.packets_retransmitted / stats.packets_sent);
}
if (stats.max_sequence_reordering == 0) if (stats.max_sequence_reordering == 0)
return; return;
const base::HistogramBase::Sample kMaxReordering = 100; const base::HistogramBase::Sample kMaxReordering = 100;
......
...@@ -31478,6 +31478,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. ...@@ -31478,6 +31478,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary> </summary>
</histogram> </histogram>
<histogram name="Net.QuicSession.PacketRetransmitsPerMille" units="permille">
<owner>ckrasic@chromium.org</owner>
<summary>
The number of packets retransmitted per 1000. Only sessions with
packets_sent &gt;= 100 are included. Recorded in session destructor.
</summary>
</histogram>
<histogram name="Net.QuicSession.PacketWriteTime"> <histogram name="Net.QuicSession.PacketWriteTime">
<owner>rch@chromium.org</owner> <owner>rch@chromium.org</owner>
<summary> <summary>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment