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-1413: Use on-disk transaction log for learner sync up


+
Thawan Kooburat 2013-05-20, 05:40
+
fpj@... 2013-05-24, 00:19
+
Thawan Kooburat 2013-05-24, 04:30
+
Thawan Kooburat 2013-05-25, 02:23
+
Raul Gutierrez Segales 2013-05-24, 22:17
+
Thawan Kooburat 2013-05-29, 06:46
+
fpj@... 2013-05-28, 21:08
+
Thawan Kooburat 2013-05-29, 19:11
+
fpj@... 2013-05-30, 14:00
+
Thawan Kooburat 2013-05-30, 18:41
+
fpj@... 2013-05-31, 07:06
+
Thawan Kooburat 2013-06-01, 00:08
+
Thawan Kooburat 2013-06-24, 08:40
+
fpj@... 2013-05-28, 21:08
+
Thawan Kooburat 2013-05-24, 22:55
Copy link to this message
-
Re: Review Request: ZOOKEEPER-1413: Use on-disk transaction log for learner sync up

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

(Updated June 24, 2013, 8:40 a.m.)
Review request for zookeeper.
Changes
-------

- Move to new SLF4J.  The new version (1.7) require JDK5.  (ZK require 1.6 at the moment).  This is required to support new method signature. I can split this into a separate jira as well if needed
- Fix other comments and refactoring
Description
-------

ZOOKEEPER-1413: Use on-disk transaction log for learner sync up

- Use txnlog for learner synchronization if learner fall too far behind
- Refactoring LearnerHandler to deal with different cases of handling learner synchronization  
This addresses bug https://issues.apache.org/jira/browse/ZOOKEEPER-1413.
    https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/ZOOKEEPER-1413
Diffs (updated)
-----

  /ivy.xml 1495522
  /src/java/main/org/apache/zookeeper/server/TxnLogProposalIterator.java PRE-CREATION
  /src/java/main/org/apache/zookeeper/server/ZKDatabase.java 1495522
  /src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java 1495522
  /src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java 1495522
  /src/java/main/org/apache/zookeeper/server/persistence/TxnLog.java 1495522
  /src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java 1495522
  /src/java/test/org/apache/zookeeper/server/quorum/LearnerHandlerTest.java PRE-CREATION
  /src/java/test/org/apache/zookeeper/test/FollowerResyncConcurrencyTest.java 1495522
  /src/java/test/org/apache/zookeeper/test/GetProposalFromTxnTest.java PRE-CREATION
  /src/java/test/org/apache/zookeeper/test/LoadFromLogTest.java 1495522

Diff: https://reviews.apache.org/r/11231/diff/
Testing
-------

- unit tests
- ran in prod for more than half a year
Thanks,

Thawan Kooburat