[PATCH 1 of 4] wireproto: add basic command batching infrastructure
timeless
timeless at gmail.com
Wed Jun 15 22:52:51 CDT 2011
On Tue, Jun 14, 2011 at 4:59 PM, Peter Arrenbrecht
<peter.arrenbrecht at gmail.com> wrote:
> + All methods invoked on instances of this class are simply queued and return a
> + a future for the result.
i think you should write 'a `future` as the result'
i'm not a fan of 'future', is it an object? i can't tell from this.
> +# demo remoting; mimicks what wireproto and HTTP/SSH do
mimics
> +# equivalent of wireproto.encode/decodelist, that is, type-specific marshalling
marshaling
> + # greet is coded directly. It therefore does not support batching. If it
> + # does appear in a batch, the batch is split around greet, and the call to
> + # greet is done in its own roundtrip.
round-trip?
More information about the Mercurial-devel
mailing list