Re: [Tarantool-patches] [PATCH 1/7] iproto: implement stream id in binary iproto protocol

[Vladimir Davydov via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

On Thu, Aug 05, 2021 at 09:17:39PM +0300, mechanik20051988 wrote:
> From: mechanik20051988 <mechanik20.05.1988@gmail.com>
> 
> For further implementation of streams, we need to separate
> requests belonging to and not belonging to streams. For this
> purpose, the stream ID field was added to the iproto binary
> protocol. For requests that do not belong to stream, this field
> is omitted or equal to zero. For requests belonging to stream,
> we use this field to determine which stream the request belongs to.
> 
> Part of #5860
> 
> @TarantoolBot document
> Title: new field in binary iproto protocol
> 
> Add new field to binary iproto protocol.
> `IPROTO_STREAM_ID 0x0a` determines whether a request
> belongs to a stream or not. If this field is omited
> or equal to zero this request doesn't belongs to stream.
> ---
>  src/box/iproto_constants.c |   4 +-
>  src/box/iproto_constants.h |   1 +
>  src/box/xrow.c             |   8 ++
>  src/box/xrow.h             |   5 ++
>  test/unit/xrow.cc          |   7 +-
>  test/unit/xrow.result      | 168 +++++++++++++++++++------------------
>  6 files changed, 109 insertions(+), 84 deletions(-)

LGTM