Page MenuHomePhabricator

Support retries in MultiHTTPClient
Closed, DeclinedPublic

Description

MultiHTTPClient currently does not support request retries out of the box, which makes connections to backend services more brittle than necessary. This is especially the case where those backend services are contacted directly via LVS, as there is no retry at all in that situation.

The retry functionality should

  • be limited to specific, configurable status codes,
  • implement exponential delay & be limited to a small number (1-2 at most).

See also

Event Timeline

I'm already looking into this, so taking this one.

Change 292909 had a related patch set uploaded (by Mobrovac):
MultiHttpClient: retry failed requests

https://gerrit.wikimedia.org/r/292909

GWicke lowered the priority of this task from High to Medium.Oct 12 2016, 10:33 PM
GWicke edited projects, added Services (blocked); removed Services.

@mobrovac are you planning to continue with this approach?

Pkra moved this task from Mathoid to Blocked: needs help on the Math board.

Change 292909 abandoned by Mobrovac:
MultiHttpClient: retry failed requests

https://gerrit.wikimedia.org/r/292909