----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/483/#review422 -----------------------------------------------------------
trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java <https://reviews.apache.org/r/483/#comment803> I have a problem here with both pingSession and touchSession. I would really like to evolve the code to be able to indicate that a follower believes one of its clients to be dead and to expire that client more quickly. However, to do that I need to be able to update the sessionTimeout to a lower value. Right now, you aren't actually passing that information into the fd. Can you update the code so that this information is preserved meaningfully? - Camille On 2011-04-09 16:20:30, Camille Fournier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/483/ > ----------------------------------------------------------- > > (Updated 2011-04-09 16:20:30) > > > 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 > 1089594 > > trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java > 1089595 > 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/InterArrivalSamplingWindowTest.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 > >
