pulkit added a comment.

  @martinvonz Thanks a lot for putting time and effort in trying alternate 
approach. Sadly this does not work. I didn't tested this on our repo yet, but I 
tested this on test-narrow-widen-no-ellipsis.t. You can also do that, by 
removing the globs from this line 
https://www.mercurial-scm.org/repo/hg-committed/file/daff528e00d7/tests/test-narrow-widen-no-ellipsis.t#l116
 and seeing the payload before and after the patch. With this patch they don't 
change, so we are sending the same amount of data before and after this patch. 
I have mentioned numbers in https://phab.mercurial-scm.org/D4887 description.

INLINE COMMENTS

> narrowwirepeer.py:96
>  
> -        bundler = bundle2.widen_bundle(repo, diffmatch, common, known,
> +        bundler = bundle2.widen_bundle(repo, diffmatch, oldmatch, common, 
> known,
>                                               cgversion, ellipses)

I swapped diffmatch and oldmatch here while testing because that's how 
widen_bundle() accepts them

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D4895

To: martinvonz, durin42, #hg-reviewers
Cc: pulkit, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to