<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 17, 2019 at 3:36 AM Pierre-Yves David <<a href="mailto:pierre-yves.david@ens-lyon.org">pierre-yves.david@ens-lyon.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On 4/17/19 5:59 AM, Martin von Zweigbergk wrote:<br>
> <br>
> <br>
> On Tue, Apr 16, 2019 at 11:41 AM Pierre-Yves David <br>
> <<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a> <mailto:<a href="mailto:pierre-yves.david@ens-lyon.org" target="_blank">pierre-yves.david@ens-lyon.org</a>>> <br>
> wrote:<br>
> <br>
>     # HG changeset patch<br>
>     # User Pierre-Yves David <<a href="mailto:pierre-yves.david@octobus.net" target="_blank">pierre-yves.david@octobus.net</a><br>
>     <mailto:<a href="mailto:pierre-yves.david@octobus.net" target="_blank">pierre-yves.david@octobus.net</a>>><br>
>     # Date 1555428398 -7200<br>
>     #      Tue Apr 16 17:26:38 2019 +0200<br>
>     # Node ID ffaa98def33a903f132ec4177d36823a741b6ef6<br>
>     # Parent  017778a4463a8e6ecb4b17cacf46a3ab27bdb239<br>
>     # EXP-Topic discovery-speedup<br>
>     # Available At <a href="https://bitbucket.org/octobus/mercurial-devel/" rel="noreferrer" target="_blank">https://bitbucket.org/octobus/mercurial-devel/</a><br>
>     #              hg pull<br>
>     <a href="https://bitbucket.org/octobus/mercurial-devel/" rel="noreferrer" target="_blank">https://bitbucket.org/octobus/mercurial-devel/</a> -r ffaa98def33a<br>
>     setdiscovery: stop limiting the number of local head we initially send<br>
> <br>
> <br>
> I think I tried something like this before. I still have a commit in my <br>
> repo. I don't remember why I didn't send it. Perhaps the issue was that <br>
> the heads are sent in HTTP headers and we need to limit the size of <br>
> those to work with restrictive proxies and servers?<br>
<br>
My intuition is that we are going to it that limit later anyway (eg: in <br>
the getbundle common parameters).<br></blockquote><div><br></div><div>IIRC, we split up the HgArgs header when it gets too big. I don't know about the getbundle parameters.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
What is our way forward here?</blockquote><div><br></div><div>Perhaps capability-guarded support for putting it in the request body? We already have support for that with experimental.httppostargs. I don't know if that also replaces GET requests by POST requests (and I don't know if the server accepts either method), so perhaps we'll need another capability here. Augie?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> This provide a large boost that I would <br>
rather have in 5.0 core instead of having to patch it all around. Should </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
we hide the behavior behind a config as a start?<br></blockquote><div><br></div><div>I'd be fine with making the limits that are currently 100 and 200 both configurable (as experimental options).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Cheers,<br>
<br>
-- <br>
Pierre-Yves David<br>
</blockquote></div></div></div></div>