-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15753/
-----------------------------------------------------------
(Updated Dec. 12, 2013, 8:30 a.m.)
Review request for zookeeper, German Blanco and fpj.
Changes
-------
I created a new patch from the latest revision (1550368). It seems that the
previous one didn't match to the changes that I made (it appeared some modified
files that I didn't modified actually). Additionally I performed the 'svn add'
operation on the test file. It should appear a new file
(src/java/test/org/apache/zookeeper/test/FLEMaxIntervalNotificationTest.java).
Thanks and regards.
Bugs: ZOOKEEPER-1814
https://issues.apache.org/jira/browse/ZOOKEEPER-1814
Repository: zookeeper
Description
-------
See Zookeeper-1814.
Diffs (updated)
-----
./src/docs/src/documentation/content/xdocs/zookeeperAdmin.xml 1550368
./src/java/main/org/apache/zookeeper/server/quorum/FastLeaderElection.java
1550368
./src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java
1550368
./src/java/test/org/apache/zookeeper/test/FLEMaxIntervalNotificationTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/15753/diff/
Testing
-------
New test case checking that the parameter maxNotificationInterval can be
configurable.
The test case overwrites the receiveConnection method in QuorumCnxManager class
in order to avoid any answer to Fast Leader Election. Thus, FLE is forced to
retry increasing exponentially the notifications interval. This interval is
measured and the test case stops when 3 consecutive measures are equal. The
test case will be OK if the measure is the configured in the cfg file.
File Attachments
----------------
ZOOKEEPER-1814.patch
https://reviews.apache.org/media/uploaded/files/2013/12/11/7e510364-6df3-486e-b338-3398960ba6b0__ZOOKEEPER-1814.patch
Thanks,
Daniel Peon
