zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vincent Poon <vincentp...@gmail.com>
Subject Review Request 57888: ZOOKEEPER-2736 Add a connection rate limiter
Date Thu, 23 Mar 2017 18:45:38 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57888/
-----------------------------------------------------------

Review request for zookeeper.


Repository: zookeeper-git


Description
-------

Currently the maxClientCnxns property only limits the aggregate number of connections from
a client, but not the rate at which connections can be created.

This patch adds a configurable connection rate limiter which limits the rate as well.


Diffs
-----

  conf/zoo_sample.cfg a5a2c0b 
  src/java/main/org/apache/zookeeper/common/RateLimiter.java PRE-CREATION 
  src/java/main/org/apache/zookeeper/common/TokenBucket.java PRE-CREATION 
  src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java 7a72757 
  src/java/main/org/apache/zookeeper/server/ZooKeeperServerBean.java 0273896 
  src/java/main/org/apache/zookeeper/server/ZooKeeperServerMXBean.java b95e265 
  src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java cb8f1c2 
  src/java/test/org/apache/zookeeper/common/TokenBucketTest.java PRE-CREATION 
  src/java/test/org/apache/zookeeper/test/ClientBase.java bb5462a 
  src/java/test/org/apache/zookeeper/test/MaxCnxnRateTest.java PRE-CREATION 
  src/java/test/org/apache/zookeeper/test/MaxCnxnsTest.java a96e5a8 


Diff: https://reviews.apache.org/r/57888/diff/1/


Testing
-------

Unit/integration tests


Thanks,

Vincent Poon


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message