[ https://issues.apache.org/jira/browse/ZOOKEEPER-2736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15939595#comment-15939595 ]
ASF GitHub Bot commented on ZOOKEEPER-2736: ------------------------------------------- Github user eribeiro commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/205#discussion_r107821391 --- Diff: src/java/test/org/apache/zookeeper/test/MaxCnxnRateTest.java --- @@ -0,0 +1,124 @@ +/** +* Licensed to the Apache Software Foundation (ASF) under one +* or more contributor license agreements. See the NOTICE file +* distributed with this work for additional information +* regarding copyright ownership. The ASF licenses this file +* to you under the Apache License, Version 2.0 (the +* "License"); you may not use this file except in compliance +* with the License. You may obtain a copy of the License at +* +* http://www.apache.org/licenses/LICENSE-2.0 +* +* Unless required by applicable law or agreed to in writing, software +* distributed under the License is distributed on an "AS IS" BASIS, +* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +* See the License for the specific language governing permissions and +* limitations under the License. +*/ + +package org.apache.zookeeper.test; + +import static org.junit.Assert.assertEquals; + +import java.util.ArrayList; +import java.util.concurrent.CountDownLatch; +import java.util.concurrent.TimeUnit; +import java.util.concurrent.atomic.AtomicInteger; + +import org.apache.zookeeper.common.TokenBucket; +import org.apache.zookeeper.server.ServerCnxnFactory; +import org.apache.zookeeper.server.quorum.QuorumPeerConfig; +import org.apache.zookeeper.test.MaxCnxnsTest.CnxnThread; +import org.junit.Test; + +public class MaxCnxnRateTest extends ClientBase { + + private int maxClientCnxnRate = 1; + private int maxClientCnxnBurst = 10; + private String host; + private int port; + static CountDownLatch startLatch; + + @Override + public void startServer() throws Exception { + LOG.info("STARTING server"); + maxCnxns = 1000; + String split[] = hostPort.split(":"); + host = split[0]; + port = Integer.parseInt(split[1]); + QuorumPeerConfig.setRateLimiterImpl(TokenBucket.class.getName()); + QuorumPeerConfig.setClientCnxnRate(maxClientCnxnRate); + QuorumPeerConfig.setClientCnxnBurst(maxClientCnxnBurst); + serverFactory = ServerCnxnFactory.createFactory(port, maxCnxns); + startServer(serverFactory); + startLatch = new CountDownLatch(1); + } + + static class LatchedCnxnThread extends CnxnThread { + + public LatchedCnxnThread(int i, String host, int port, AtomicInteger connectionCounter) { + super(i, host, port, connectionCounter); + } + + @Override + public void run() { + try { + startLatch.await(); + super.run(); + --- End diff -- Unnecessary blank line. > Add a connection rate limiter > ----------------------------- > > Key: ZOOKEEPER-2736 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2736 > Project: ZooKeeper > Issue Type: Improvement > Components: server > Affects Versions: 3.4.9, 3.5.2 > Reporter: Vincent Poon > Attachments: ZOOKEEPER-2736.v1.patch > > > 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. -- This message was sent by Atlassian JIRA (v6.3.15#6346)