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

Issue 710828 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android
Pri: 2
Type: Bug



Sign in to add a comment

[Remote suggestions] Report time distribution of triggers

Project Member Reported by jkrcal@chromium.org, Apr 12 2017

Issue description

When changing scheduling intervals for soft background fetches, we need to guess blindly what the impact on traffic this will have.

We need to record the distribution of:
 - when first trigger comes (not necessarily successful), and
 - when the (first) successful trigger comes.

The former helps us to evaluate impact of scheduling changes. The latter helps us to understand what are the real fetching intervals in the wild.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/51a10acb782ef180630115d21190b7c70cb44944

commit 51a10acb782ef180630115d21190b7c70cb44944
Author: jkrcal <jkrcal@chromium.org>
Date: Thu Apr 13 23:09:13 2017

[Remote suggestions] Report time distribution of triggers

When changing scheduling intervals for soft background fetches, we had
to guess blindly what the impact on traffic this will have.

This CL records the distribution of:
 - when first trigger comes (not necessarily successful), and
 - when the (first) successful trigger comes.

The former helps us to evaluate impact of scheduling changes. The latter
helps us to understand what are the real fetching intervals in the wild.

BUG= 710828 

Review-Url: https://codereview.chromium.org/2810053004
Cr-Commit-Position: refs/heads/master@{#464595}

[modify] https://crrev.com/51a10acb782ef180630115d21190b7c70cb44944/components/ntp_snippets/remote/remote_suggestions_scheduler_impl.cc
[modify] https://crrev.com/51a10acb782ef180630115d21190b7c70cb44944/components/ntp_snippets/remote/remote_suggestions_scheduler_impl.h
[modify] https://crrev.com/51a10acb782ef180630115d21190b7c70cb44944/tools/metrics/histograms/histograms.xml

Comment 2 by jkrcal@chromium.org, Apr 18 2017

Status: Fixed (was: Started)

Sign in to add a comment