RESOLVED FIXED33412
Retry writes to QueueStatusServer when we get 500 errors 
https://bugs.webkit.org/show_bug.cgi?id=33412
Summary Retry writes to QueueStatusServer when we get 500 errors
Adam Barth
Reported 2010-01-08 17:56:14 PST
Retry writes to QueueStatusServer when we get 500 errors
Attachments
Patch (3.59 KB, patch)
2010-01-08 17:57 PST, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch (10.94 KB, patch)
2010-01-09 16:09 PST, Adam Barth 		eric: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2010-01-08 17:57:23 PST
Created attachment 46186 [details] Patch
Eric Seidel (no email)
Comment 2 2010-01-09 14:56:01 PST
Comment on attachment 46186 [details] Patch I think this ability should be made general (i'm sure we'll need it in other places). I also think this should eventually stop. I don't think we should hang the queue forever if AppEngine is really wedged, or? A general method would take some sort of closure and have some default backoff amount, and eventual "give-up" timeout, no?
Eric Seidel (no email)
Comment 3 2010-01-09 14:56:56 PST
Comment on attachment 46186 [details] Patch This block of code is actually huge as is. So breaking it down into (a) function(s), whether for use with some helper retry-thingy or not, seems useful.
Adam Barth
Comment 4 2010-01-09 16:09:54 PST
Created attachment 46226 [details] Patch
Eric Seidel (no email)
Comment 5 2010-01-09 16:15:23 PST
Comment on attachment 46226 [details] Patch So hot.
Note You need to log in before you can comment on or make changes to this bug.