[issue3318] getbundle heads and common lists can be redundant and large

Kevin Gessner bugs at mercurial.selenic.com
Mon Mar 12 15:39:30 CDT 2012


New submission from Kevin Gessner <kevin+bts at fogcreek.com>:

I have a Kiln customer with a 688 head repo. Trying to pull causes errors 
because getbundle needs 58kb of headers for common= and heads=, which breaks 
our proxy.

I can't provide the actual repo, but I can provide the header contents and 
possible some other repo data.

Quoth IRC:

(16:30:19) mpm1: If the head says "x is in common", why does it also say "send 
me everything between common and x"?
(16:31:01) tonfa: remote heads?
(16:31:15) mpm1: getbundle(heads, common) is basically "all changesets ::heads 
- ::common", right?
(16:31:24) tonfa: yes
(16:32:31) mpm1: Then it seems to me that sending an element in heads that is 
already in common is redundant.
(16:33:18) tonfa: it can safely be omitted from heads

----------
messages: 19360
nosy: kevingessner, mpm, tonfa
priority: bug
status: unread
title: getbundle heads and common lists can be redundant and large

____________________________________________________
Mercurial issue tracker <bugs at mercurial.selenic.com>
<http://mercurial.selenic.com/bts/issue3318>
____________________________________________________


More information about the Mercurial-devel mailing list