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/11/11 12:35:43 UTC

[GitHub] [incubator-ratis] szetszwo opened a new pull request #269: RATIS-1148. In NettyServerStreamRpc, the remote writes should not wait for the previous reply.

szetszwo opened a new pull request #269:
URL: https://github.com/apache/incubator-ratis/pull/269


   See https://issues.apache.org/jira/browse/RATIS-1148


----------------------------------------------------------------
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



[GitHub] [incubator-ratis] runzhiwang commented on pull request #269: RATIS-1148. In NettyServerStreamRpc, the remote writes should not wait for the previous reply.

Posted by GitBox <gi...@apache.org>.
runzhiwang commented on pull request #269:
URL: https://github.com/apache/incubator-ratis/pull/269#issuecomment-725401115


   @szetszwo Hi, we have discussed it previously in pr: https://github.com/apache/incubator-ratis/pull/250.
   ![image](https://user-images.githubusercontent.com/51938049/98812763-ff606d00-245d-11eb-98b0-e5208b9d8a07.png)
   


----------------------------------------------------------------
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



[GitHub] [incubator-ratis] runzhiwang merged pull request #269: RATIS-1148. In NettyServerStreamRpc, each remote stream should not wait for its own previous write.

Posted by GitBox <gi...@apache.org>.
runzhiwang merged pull request #269:
URL: https://github.com/apache/incubator-ratis/pull/269


   


----------------------------------------------------------------
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



[GitHub] [incubator-ratis] szetszwo commented on pull request #269: RATIS-1148. In NettyServerStreamRpc, each remote stream should not wait for its own previous write.

Posted by GitBox <gi...@apache.org>.
szetszwo commented on pull request #269:
URL: https://github.com/apache/incubator-ratis/pull/269#issuecomment-725407725


   @runzhiwang , I might have misunderstood your previous comment since Netty uses multiple threads to call channelRead(..).  I thought you were saying that it must require the same thread submitting the calls.  Sorry.
   
   You are right that OrderedStreamAsync will take care the async calls even if there are multithreads calling OrderedStreamAsync.


----------------------------------------------------------------
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