Code review comment for lp:~mwhudson/launchpad/puller-doesnt-call-startMirroring

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

> Looks good.
>
> I also noticed your comment on IRC about how you thought it may be possible to
> have two workers pulling the same branch. In normal circumstances it won't
> be, however I just thought of an edge case where it is currently pulling a
> branch, and a new requestMirror comes in, will another worker try to pull it
> again?

Yeah, that's exactly what I mean.

Cheers,
mwh

« Back to merge proposal