> On 2011-03-08 16:01:30, Camille Fournier wrote: > > trunk/src/java/test/org/apache/zookeeper/test/fd/BertierFDTest.java, line 29 > > <https://reviews.apache.org/r/483/diff/1/?file=13841#file13841line29> > > > > A rather large nit to pick: I wish the asserts in your tests had a > > message associated with them. If/When it fails, it would be nice to have a > > bit of context to know what we were testing and why
Right. I will add assertion messages to all the FD tests. It will be addressed in the next patch. - Abmar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/483/#review316 ----------------------------------------------------------- On 2011-03-08 15:52:08, Camille Fournier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/483/ > ----------------------------------------------------------- > > (Updated 2011-03-08 15:52:08) > > > Review request for zookeeper. > > > Summary > ------- > > https://issues.apache.org/jira/browse/ZOOKEEPER-702 > > > Diffs > ----- > > trunk/src/docs/src/documentation/content/xdocs/index.xml 1065709 > trunk/src/docs/src/documentation/content/xdocs/zookeeperFailureDetector.xml > PRE-CREATION > trunk/src/java/main/org/apache/zookeeper/ClientCnxn.java 1065709 > trunk/src/java/main/org/apache/zookeeper/ClientCnxnSocket.java 1065709 > trunk/src/java/main/org/apache/zookeeper/ClientCnxnSocketNIO.java 1065709 > trunk/src/java/main/org/apache/zookeeper/ZooKeeper.java 1065709 > trunk/src/java/main/org/apache/zookeeper/ZooKeeperMain.java 1065709 > > trunk/src/java/main/org/apache/zookeeper/common/fd/BertierFailureDetector.java > PRE-CREATION > trunk/src/java/main/org/apache/zookeeper/common/fd/ChenFailureDetector.java > PRE-CREATION > trunk/src/java/main/org/apache/zookeeper/common/fd/FailureDetector.java > PRE-CREATION > > trunk/src/java/main/org/apache/zookeeper/common/fd/FailureDetectorFactory.java > PRE-CREATION > > trunk/src/java/main/org/apache/zookeeper/common/fd/FailureDetectorOptParser.java > PRE-CREATION > > trunk/src/java/main/org/apache/zookeeper/common/fd/FixedPingFailureDetector.java > PRE-CREATION > > trunk/src/java/main/org/apache/zookeeper/common/fd/InterArrivalSamplingWindow.java > PRE-CREATION > trunk/src/java/main/org/apache/zookeeper/common/fd/MessageType.java > PRE-CREATION > > trunk/src/java/main/org/apache/zookeeper/common/fd/PhiAccrualFailureDetector.java > PRE-CREATION > > trunk/src/java/main/org/apache/zookeeper/common/fd/SlicedPingFailureDetector.java > PRE-CREATION > trunk/src/java/main/org/apache/zookeeper/server/ServerConfig.java 1065709 > trunk/src/java/main/org/apache/zookeeper/server/SessionTracker.java 1065709 > trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java > 1065709 > trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java > 1065709 > trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServerMain.java > 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerZooKeeperServer.java > 1065709 > trunk/src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/LeaderZooKeeperServer.java > 1065709 > trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java 1072085 > trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java > 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerSessionTracker.java > 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerZooKeeperServer.java > 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/ObserverZooKeeperServer.java > 1065709 > trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java > 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java > 1065709 > trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java > 1065709 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumZooKeeperServer.java > 1065709 > trunk/src/java/test/org/apache/zookeeper/TestableZooKeeper.java 1065709 > trunk/src/java/test/org/apache/zookeeper/test/ClientBase.java 1065709 > trunk/src/java/test/org/apache/zookeeper/test/DisconnectableZooKeeper.java > 1065709 > trunk/src/java/test/org/apache/zookeeper/test/QuorumBase.java 1065709 > trunk/src/java/test/org/apache/zookeeper/test/QuorumFDHammerTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/RecoveryTest.java 1065709 > trunk/src/java/test/org/apache/zookeeper/test/SessionTest.java 1065709 > > trunk/src/java/test/org/apache/zookeeper/test/fd/BertierClientHammerTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/BertierFDTest.java > PRE-CREATION > > trunk/src/java/test/org/apache/zookeeper/test/fd/BertierQuorumHammerTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/BertierRecoveryTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/BertierSessionTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/ChenClientHammerTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/ChenFDTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/ChenQuorumHammerTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/ChenRecoveryTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/ChenSessionTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/FixedPingFDTest.java > PRE-CREATION > > trunk/src/java/test/org/apache/zookeeper/test/fd/PhiAccrualClientHammerTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/PhiAccrualFDTest.java > PRE-CREATION > > trunk/src/java/test/org/apache/zookeeper/test/fd/PhiAccrualQuorumHammerTest.java > PRE-CREATION > > trunk/src/java/test/org/apache/zookeeper/test/fd/PhiAccrualRecoveryTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/PhiAccrualSessionTest.java > PRE-CREATION > trunk/src/java/test/org/apache/zookeeper/test/fd/SlicedPingFDTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/483/diff > > > Testing > ------- > > unit tests > > > Thanks, > > Camille > >
