You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ratis.apache.org by GitBox <gi...@apache.org> on 2022/03/10 06:06:40 UTC

[GitHub] [ratis] guohao-rosicky opened a new pull request #621: RATIS-1530. Cleanup NettyClientStreamRpc#replies empty queue optimize

guohao-rosicky opened a new pull request #621:
URL: https://github.com/apache/ratis/pull/621


   ## What changes were proposed in this pull request?
   
   Cleanup NettyClientStreamRpc#replies empty queue optimize
   
   
   
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/RATIS-1530


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

To unsubscribe, e-mail: issues-unsubscribe@ratis.apache.org

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

[GitHub] [ratis] szetszwo commented on pull request #621: RATIS-1530. Cleanup NettyClientStreamRpc#replies empty queue optimize

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


   @guohao-rosicky , it seems that we won't get much benefit when optimizing the case when the queue is empty (i.e. when there are no requests.)
   
   Also, running tasks in the background actually may make the worst case (when there are many requests) more worst.  What do you think?


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

To unsubscribe, e-mail: issues-unsubscribe@ratis.apache.org

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

[GitHub] [ratis] guohao-rosicky commented on pull request #621: RATIS-1530. Cleanup NettyClientStreamRpc#replies empty queue optimize

Posted by GitBox <gi...@apache.org>.
guohao-rosicky commented on pull request #621:
URL: https://github.com/apache/ratis/pull/621#issuecomment-1063699673


   What do you think of this feature? @szetszwo


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

To unsubscribe, e-mail: issues-unsubscribe@ratis.apache.org

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