Closed Bug 1261206 Opened 5 years ago Closed 5 years ago

Add redo logic to pushlog_client

Categories

(Developer Services :: Mercurial: Pushlog, defect)

Product:
Developer Services
Component:
Mercurial: Pushlog
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: armenzg, Assigned: sabergeass)

Details

Attachments

(1 file)

Pr for bug 1261206
55 bytes, text/x-github-pull-request
armenzg
: review+
Details | Review 
We currently don't retry to query pushlog.

We can use redo to improve this:
https://github.com/bhearsum/redo
Attached file Pr for bug 1261206 — 
Hi armen, does it looks good to you?
Assignee: nobody → sabergeass
Status: NEW → ASSIGNED
Attachment #8737048 - Flags: review?(armenzg)
Comment on attachment 8737048 [details] [review]
Pr for bug 1261206

I will land this and make a new release.
Attachment #8737048 - Flags: review?(armenzg) → review+
Flags: needinfo?(armenzg)
Hi MikeLing,
For following bugs, would you mind if we tried using Mercurial and MozReview? It makes it easier for me to land changes faster.
Flags: needinfo?(armenzg)
(In reply to Armen Zambrano [:armenzg] - Engineering productivity from comment #3)
> Hi MikeLing,
> For following bugs, would you mind if we tried using Mercurial and
> MozReview? It makes it easier for me to land changes faster.

not at all, though I'm mot familiar with the Mercurial, but I will try my best on it :)
I believe we could close this. :)
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.