D6267: incoming: detect if server send partial replies

joerg.sonnenberger (Joerg Sonnenberger) phabricator at mercurial-scm.org
Wed Jul 17 14:02:34 UTC 2019


joerg.sonnenberger added inline comments.
joerg.sonnenberger marked 2 inline comments as done.

INLINE COMMENTS

> indygreg wrote in bundlerepo.py:642
> Something I didn't notice before is that this function doesn't do any UI presentation. So it feels like a bug to print a message in this function assuming that fetched changesets will be displayed.
> 
> I suppose that means we'll need to include the "was partial reply" state in the return value and format it to a warning elsewhere.
> 
> Sorry for not catching this on first review. I intend to queue this patch for stable so it gets in the 5.0 release, as I think the improved UI is good to have.

We know that this whole code fragment for "incoming" is seriously limited and should go away. Do we really want to change the whole API for that one special use case?

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6267/new/

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

To: joerg.sonnenberger, #hg-reviewers, indygreg
Cc: indygreg, mercurial-devel


More information about the Mercurial-devel mailing list