You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2019/09/06 12:11:55 UTC

[GitHub] [zookeeper] anmolnar edited a comment on issue #1037: ZOOKEEPER-3492: Add weights to server side connection throttling

anmolnar edited a comment on issue #1037: ZOOKEEPER-3492: Add weights to server side connection throttling
URL: https://github.com/apache/zookeeper/pull/1037#issuecomment-528830028
 
 
   @jhuan31 Thanks for your patches related to throttling. I think this is a very useful addition to ZooKeeper. Also great to see you added the new config settings to the docs.
   
   However, given the size and impact of the feature, I'd really love to see a dedicated section in the documentation for throttling. Something similar to what I added for Quorum TLS: https://github.com/apache/zookeeper/blob/master/zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md#Quorum+TLS
   
   It'd be really nice to see detailed explanation of how this Blue-throttling stuff works, where does it come from, some links to literature for instance, etc. I also imagine a nice how-to about the right way of setting things up.
   
   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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services