Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29322734: Issue 284 - Unify initial delay for all downloads (Closed)

Created:
July 22, 2015, 12:54 p.m. by Wladimir Palant
Modified:
July 25, 2015, 11:12 p.m.
Reviewers:
Felix Dahlke
Visibility:
Public.

Description

Issue 284 - Unify initial delay for all downloads

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M lib/synchronizer.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
Wladimir Palant
July 22, 2015, 12:54 p.m. (2015-07-22 12:54:39 UTC) #1
Felix Dahlke
LGTM. Although now that the delays for notification and subscription downloads are the same, I'm ...
July 22, 2015, 1 p.m. (2015-07-22 13:00:05 UTC) #2
Felix Dahlke
July 22, 2015, 1:01 p.m. (2015-07-22 13:01:16 UTC) #3
On 2015/07/22 13:00:05, Felix Dahlke wrote:
> LGTM. Although now that the delays for notification and subscription downloads
> are the same, I'm wondering if we shouldn't just always have one minute as
> initial delay in downloader.

Then again, it's not the only parameter that's identical, never mind.

Powered by Google App Engine
This is Rietveld