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

Issue 5056003: Retry in parallel_emerge for 5xx error codes. (Closed)

Created:
10 years, 1 month ago by davidjames
Modified:
9 years, 4 months ago
Reviewers:
petkov, sosa
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Retry in parallel_emerge for 5xx error codes. When the server returns a 5xx error code, we should retry. Such errors are often due to flakiness, so retries resolve the issue. BUG=chromium-os:9217 TEST=Tested parallel_emerge with fake URL to verify it retries. Change-Id: Ie5cf003562b5261005be9929624769394b4802a1 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=8ce0100

Patch Set 1 #

Total comments: 2

Patch Set 2 : Add sleeps in between retries per petkov's comments. #

Patch Set 3 : Address sosa's comments. #

Patch Set 4 : Add explicit conversion for better error messages. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -5 lines) Patch
M parallel_emerge View 1 2 3 2 chunks +11 lines, -5 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
davidjames
10 years, 1 month ago (2010-11-16 18:07:12 UTC) #1
sosa
http://codereview.chromium.org/5056003/diff/1/parallel_emerge File parallel_emerge (right): http://codereview.chromium.org/5056003/diff/1/parallel_emerge#newcode686 parallel_emerge:686: if i + 1 == tries or e.code < ...
10 years, 1 month ago (2010-11-16 18:09:25 UTC) #2
petkov
10 years, 1 month ago (2010-11-16 18:17:33 UTC) #3
LGTM but you should change == tries to >= tries. And you may want to change "i"
to "try". Thanks!

Powered by Google App Engine
This is Rietveld 408576698