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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/05/31 16:19:22 UTC

[GitHub] [pulsar] 315157973 commented on pull request #10754: Support max-connection and max-connection-per-IP

315157973 commented on pull request #10754:
URL: https://github.com/apache/pulsar/pull/10754#issuecomment-851583746


   > I have some concerns, if connections reach the limitation, the rejected client will re-connect many times and this may confuse users.
   > 
   > Maybe we could make clients receive this connection limit exception and cancel the retry operation, this will also make users know why the client failed to connect with the broker.
   
    I will add an exception to prevent the client from retrying. However, clients with lower versions will try to reconnect and eventually time out.


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