You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2020/10/23 18:30:56 UTC

[GitHub] [incubator-ratis] amaliujia commented on pull request #231: RATIS-1105. Refactor Netty streaming encoder and decoder.

amaliujia commented on pull request #231:
URL: https://github.com/apache/incubator-ratis/pull/231#issuecomment-715505860


   @runzhiwang  thanks for identifying the out-of-order reply.
   
   @szetszwo it sounds surprising that the ordering sounds like was guaranteed by the order of replies so potentially because faster code execution, replies could become out-of-order. I thought the ordering is maintained by `orderedStreamAsync`. E.g. there is a sequence number so a reply is available only when all lower sequence numbers have replies. Maybe this is not the case in Ratis.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org