Yakov, >This should be set up in handshake and used for all requests made over that connection. Sounds reasonable.
On Wed, Aug 2, 2017 at 6:29 PM, Yakov Zhdanov <yzhda...@apache.org> wrote: > > Response should contains Node ID that send response to thin client. > > This should be set up in handshake and used for all requests made over that > connection. > > --Yakov > > 2017-08-02 14:12 GMT+03:00 Alexey Kuznetsov <akuznet...@apache.org>: > > > Pavel, > > > > I remember one thing that is very useful for thing client: > > Response should contains Node ID that send response to thin client. > > > > This can be used for debugging needs. > > > > On Wed, Aug 2, 2017 at 4:10 PM, Yakov Zhdanov <yzhda...@apache.org> > wrote: > > > > > Pavel, > > > > > > 2. Disagree here. In your sample responses contain only success flag > > which > > > is byte and 4 bytes length. We can easily avoid that just sending > > > OP_CODE_SUCCESS or OP_CODE_FAILURE. > > > 3. In and out where applicable. > > > > > > --Yakov > > > > > > > > > > > -- > > Alexey Kuznetsov > > > -- Alexey Kuznetsov
