eolivelli commented on a change in pull request #238: Issue-237 Upgrade to ZooKeeper 3.5.3-BETA URL: https://github.com/apache/bookkeeper/pull/238#discussion_r127816511
########## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/test/ZooKeeperUtil.java ########## @@ -47,6 +47,12 @@ import static org.junit.Assert.*; public class ZooKeeperUtil { + + static { Review comment: @sijie for 3.5.3 most of four letter words commands are disabled by default in order to prevent DoS attacks. We are using ClientBase to wait for ZK server to start and this uses the fourbletter words API. Without this change tests will be very slow because the wait will always timeout ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services