-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8935/
-----------------------------------------------------------

Review request for zookeeper, Patrick Hunt and Mahadev Konar.


Description
-------

This patch add support for local session to ZooKeeper. 


This addresses bug ZOOKEEPER-1147.
    https://issues.apache.org/jira/browse/ZOOKEEPER-1147


Diffs
-----

  /src/java/main/org/apache/zookeeper/KeeperException.java 1427034 
  /src/java/main/org/apache/zookeeper/cli/CreateCommand.java 1427034 
  /src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java 1427034 
  /src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java 1427034 
  /src/java/main/org/apache/zookeeper/server/Request.java 1427034 
  /src/java/main/org/apache/zookeeper/server/SessionTracker.java 1427034 
  /src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java 1427034 
  /src/java/main/org/apache/zookeeper/server/TraceFormatter.java 1427034 
  /src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java 1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java 
1427034 
  
/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java 
1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/LeaderSessionTracker.java 
PRE-CREATION 
  /src/java/main/org/apache/zookeeper/server/quorum/LeaderZooKeeperServer.java 
1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/Learner.java 1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java 1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/LearnerSessionTracker.java 
1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/LearnerZooKeeperServer.java 
1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/LocalSessionTracker.java 
PRE-CREATION 
  
/src/java/main/org/apache/zookeeper/server/quorum/ObserverRequestProcessor.java 
1427034 
  
/src/java/main/org/apache/zookeeper/server/quorum/ProposalRequestProcessor.java 
1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java 
1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java 1427034 
  /src/java/main/org/apache/zookeeper/server/quorum/QuorumZooKeeperServer.java 
1427034 
  
/src/java/main/org/apache/zookeeper/server/quorum/ReadOnlyZooKeeperServer.java 
1427034 
  
/src/java/main/org/apache/zookeeper/server/quorum/UpgradeableSessionTracker.java
 PRE-CREATION 
  /src/java/test/org/apache/zookeeper/server/PrepRequestProcessorTest.java 
1427034 
  /src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java 
1427034 
  /src/java/test/org/apache/zookeeper/test/LeaderSessionTrackerTest.java 
PRE-CREATION 
  /src/java/test/org/apache/zookeeper/test/LocalSessionRequestTest.java 
PRE-CREATION 
  /src/java/test/org/apache/zookeeper/test/LocalSessionsOnlyTest.java 
PRE-CREATION 
  /src/java/test/org/apache/zookeeper/test/QuorumBase.java 1427034 
  /src/java/test/org/apache/zookeeper/test/QuorumTest.java 1427034 
  /src/java/test/org/apache/zookeeper/test/QuorumUtil.java 1427034 
  /src/java/test/org/apache/zookeeper/test/ReadOnlyModeTest.java 1427034 
  /src/java/test/org/apache/zookeeper/test/SessionTrackerCheckTest.java 
PRE-CREATION 
  /src/java/test/org/apache/zookeeper/test/SessionUpgradeTest.java PRE-CREATION 
  /src/java/test/org/apache/zookeeper/test/WatcherTest.java 1427034 

Diff: https://reviews.apache.org/r/8935/diff/


Testing
-------

unit tests.  For our internal branch, we haven't run into bugs related to local 
session for quite a while so it is quite stable. We enabled local session by 
default in all our deployment.


Thanks,

Thawan Kooburat

Reply via email to