# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 2018-08-22 17:29:30
# Node ID 1467b6c27ff929dc9601ad38d2f8bd4a259f08a3
# Parent  4971c97242063e8bf3f8a8496837a0db5fce123f

wireprotov2peer: add TODO about streaming responses

This is a pretty big deficiency in the current implementation.
We don't want to forget about this.

Differential Revision: https://phab.mercurial-scm.org/D4435

diff --git a/mercurial/wireprotov2peer.py b/mercurial/wireprotov2peer.py
--- a/mercurial/wireprotov2peer.py
+++ b/mercurial/wireprotov2peer.py
@@ -136,6 +136,9 @@ class clienthandler(object):
         response = self._responses[frame.requestid]
 
         if action == 'responsedata':
+            # This buffers all data until end of stream is received. This
+            # is bad for performance.
+            # TODO make response data streamable
             response.b.write(meta['data'])
 
             if meta['eos']: