[PATCH] bundle: don't send too many changesets (Issue1704)

Nicolas Dumazet nicdumz at gmail.com
Fri Nov 6 18:36:08 CST 2009


2009/11/7 Peter Arrenbrecht <peter.arrenbrecht at gmail.com>:

[...]

> The fast path in changegroupsubset can send too many csets.

> The fix is to not use findmissing(), but use nodesbetween() instead, as
> do the slow path and incoming/outgoing.
>

>         if revs is None:
>             # use the fast path, no race possible on push

>         if extranodes is None:
>             # can we go through the fast path ?


I'm not too sure, but perhaps the comments need to be updated here?

-- 
Nicolas Dumazet — NicDumZ



More information about the Mercurial-devel mailing list