Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(55)

Issue 17468009: Increase sync jobs on bots to 4 (Closed)

Created:
7 years, 6 months ago by Isaac (away)
Modified:
7 years, 6 months ago
Reviewers:
cmp
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org, johnw1
Visibility:
Public.

Description

Increase sync jobs on bots to 4 This will speed up main waterfall sync. We can remove this hack tomorrow if svn load is still OK. TBR=cmp Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=207277

Patch Set 1 #

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

Messages

Total messages: 6 (0 generated)
Isaac (away)
Will wait until all clear from John before landing.
7 years, 6 months ago (2013-06-20 01:56:15 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ilevy@chromium.org/17468009/1
7 years, 6 months ago (2013-06-20 02:33:58 UTC) #2
commit-bot: I haz the power
Change committed as 207277
7 years, 6 months ago (2013-06-20 02:35:39 UTC) #3
cmp
lgtm, should we drop this completely now and get back to the original code?
7 years, 6 months ago (2013-06-21 06:52:11 UTC) #4
Isaac (away)
Not sure; this doesn't seem to be affecting sync very much and puts lower load ...
7 years, 6 months ago (2013-06-21 06:55:08 UTC) #5
cmp
7 years, 6 months ago (2013-06-21 07:04:48 UTC) #6
Message was sent while issue was closed.
On 2013/06/21 06:55:08, Isaac wrote:
> Not sure; this doesn't seem to be affecting sync very much and puts lower load
> on our svn servers.  We could keep until we switch to git.

I would not be surprised if higher job counts start to cost us more in terms of
swapping/concurrent disk access.  Fine to leave it as-is, IMO.

Powered by Google App Engine
This is Rietveld 408576698