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

Issue 628743 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Adaptive wait time between successive component updates

Project Member Reported by sorin@chromium.org, Jul 15 2016

Issue description

The current implementation of the component updater uses a compile-time constant for the time interval it waits between applying successive component updates, when the update response includes updates to several components. The current wait time is 15 minutes.

The component updates are applied sequentially, in a priority order determined by the the order in which components are registered with the component update service.

This algorithm could delay applying updates for components that are queued behind other components which update frequently (daily basis) but the cost of those updates is small.

A change in the update algorithm is suggested, so that the wait time between components is adaptive, and it depends on the cost of the previous component update.

* Update-check all components 6 minutes after browser start.
* Apply the first update immediately; let T be the time this takes.
* for each remaining update {
    * Wait T or 15 minutes, whichever is less.
    * Apply the update.
    * Let T for the next update be the time it took to apply this update.
}
* Wait 6 hours then repeat this process.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 15 2016

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

commit 1918f9fe5780b533e69592c6c4450eee8ab2d8f1
Author: sorin <sorin@chromium.org>
Date: Fri Jul 15 20:45:04 2016

Adaptive delay between successive component updates.

The idea is to have an adaptive wait time between component updates with is determined by how long the previous update took.

R=waffles
BUG= 628743 

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

[modify] https://crrev.com/1918f9fe5780b533e69592c6c4450eee8ab2d8f1/components/update_client/action.cc
[modify] https://crrev.com/1918f9fe5780b533e69592c6c4450eee8ab2d8f1/components/update_client/crx_update_item.h

Comment 2 by sorin@chromium.org, Jul 15 2016

Status: Fixed (was: Started)

Sign in to add a comment