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

Switch to Plain View
Zookeeper >> mail # dev >> Re: Review Request: Dynamic reconfiguration, see https://issues.apache.org/jira/browse/ZOOKEEPER-107


+
Edward Ribeiro 2013-01-23, 07:30
+
Alexander Shraer 2013-01-25, 07:43
Copy link to this message
-
Re: Review Request: Dynamic reconfiguration, see https://issues.apache.org/jira/browse/ZOOKEEPER-107

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

/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java
<https://reviews.apache.org/r/6707/#comment33709>

    Lines 467-479 contains duplicate code. I suggest to create a method split (or parse) method like:
    
    List<String> split(String input) {
    
       if (input != null) {
    
         List<String> list = Arrays.asList(input.split(","));
    
         return Collections.unmodifiableList(list);
       }
    
       return Collections.emptyList();
    }
    
    Then lines 467 and 474 can be simplified as:
    
    List<String> joiningServers = split(reconfigRequest.getJoiningServers());
    
    List<String> leavingServers = split(reconfigRequest.getLeavingServers());
    
    PS: I think this split method should go in a helper class as stated before.
- Edward Ribeiro
On Nov. 29, 2012, 7:12 a.m., Alexander Shraer wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6707/
> -----------------------------------------------------------
>
> (Updated Nov. 29, 2012, 7:12 a.m.)
>
>
> Review request for zookeeper.
>
>
> Description
> -------
>
> see https://issues.apache.org/jira/browse/ZOOKEEPER-107
>
>
> Diffs
> -----
>
>   /src/c/include/proto.h 1415037
>   /src/c/include/zookeeper.h 1415037
>   /src/c/src/cli.c 1415037
>   /src/c/src/zookeeper.c 1415037
>   /src/java/main/org/apache/zookeeper/KeeperException.java 1415037
>   /src/java/main/org/apache/zookeeper/ZooDefs.java 1415037
>   /src/java/main/org/apache/zookeeper/ZooKeeper.java 1415037
>   /src/java/main/org/apache/zookeeper/ZooKeeperMain.java 1415037
>   /src/java/main/org/apache/zookeeper/cli/GetConfigCommand.java PRE-CREATION
>   /src/java/main/org/apache/zookeeper/cli/ReconfigCommand.java PRE-CREATION
>   /src/java/main/org/apache/zookeeper/server/DataTree.java 1415037
>   /src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java 1415037
>   /src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java 1415037
>   /src/java/main/org/apache/zookeeper/server/NettyServerCnxnFactory.java 1415037
>   /src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java 1415037
>   /src/java/main/org/apache/zookeeper/server/Request.java 1415037
>   /src/java/main/org/apache/zookeeper/server/ServerCnxnFactory.java 1415037
>   /src/java/main/org/apache/zookeeper/server/TraceFormatter.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/FastLeaderElection.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/Follower.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/Learner.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/Observer.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/ObserverRequestProcessor.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/QuorumBean.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/QuorumZooKeeperServer.java 1415037
>   /src/java/main/org/apache/zookeeper/server/quorum/ReadOnlyRequestProcessor.java 1415037
+
Alexander Shraer 2013-01-25, 07:43
+
Edward Ribeiro 2013-01-23, 07:35
+
Edward Ribeiro 2013-01-23, 07:32
+
Edward Ribeiro 2013-01-23, 07:08
+
Alexander Shraer 2013-01-25, 07:43
+
Edward Ribeiro 2013-01-23, 06:57
+
Alexander Shraer 2013-01-25, 07:59
+
fpj@... 2013-01-31, 00:58
+
Alexander Shraer 2013-01-31, 17:05
+
Alexander Shraer 2013-02-05, 05:07
+
Alexander Shraer 2013-02-06, 01:00
+
Edward Ribeiro 2013-03-22, 23:32