Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Zookeeper, mail # dev - Review Request: ZOOKEEPER-1147: Add support for local sessions


Copy link to this message
-
Review Request: ZOOKEEPER-1147: Add support for local sessions
Thawan Kooburat 2013-01-12, 04:27

-----------------------------------------------------------
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

+
Thawan Kooburat 2013-01-14, 23:48
+
Edward Ribeiro 2013-01-24, 21:49
+
Thawan Kooburat 2013-01-28, 08:00
+
Edward Ribeiro 2013-01-27, 23:58
+
Thawan Kooburat 2013-01-28, 08:00
+
Thawan Kooburat 2013-02-20, 01:00
+
Thawan Kooburat 2013-02-20, 02:20
+
Thawan Kooburat 2013-03-02, 20:03
+
Thawan Kooburat 2013-03-06, 23:42
+
Edward Ribeiro 2013-03-23, 01:07
+
Edward Ribeiro 2013-03-23, 00:57
+
Edward Ribeiro 2013-03-22, 23:54
+
Edward Ribeiro 2013-03-23, 01:00
+
Edward Ribeiro 2013-03-22, 23:59
+
Edward Ribeiro 2013-03-22, 23:42
+
Edward Ribeiro 2013-03-22, 23:41
+
Edward Ribeiro 2013-03-22, 23:23
+
Thawan Kooburat 2013-04-24, 01:31