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 12983: ZOOKEEPER-1691


+
Helen Hastings 2013-08-19, 21:26
+
Edward Ribeiro 2013-08-20, 00:39
+
Helen Hastings 2013-08-21, 22:09
Copy link to this message
-
Re: Review Request 12983: ZOOKEEPER-1691
Edward Ribeiro 2013-08-20, 02:00

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

http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/test/org/apache/zookeeper/server/quorum/StandaloneDisabledTest.java
<https://reviews.apache.org/r/12983/#comment49716>

    This method is obnoxiously large! Please, avoid large method bodies as those. I know you'll find plenty of large method bodies in ZK code base already, but help us to avoid its further proliferation. For example, try to extract chunks of functionality like lines 62-68 into its own private methods. Or break this large test methods into smaller ones. This helps readability and maintainability.
- Edward Ribeiro
On Aug. 19, 2013, 9:26 p.m., Helen Hastings wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12983/
> -----------------------------------------------------------
>
> (Updated Aug. 19, 2013, 9:26 p.m.)
>
>
> Review request for zookeeper and Alexander Shraer.
>
>
> Repository: zookeeper
>
>
> Description
> -------
>
> Added standaloneEnabled flag to QuorumPeerConfig to give the option to disable standalone mode for a single server.
>
>
> Diffs
> -----
>
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/docs/src/documentation/content/xdocs/zookeeperAdmin.xml 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/cli/ReconfigCommand.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FastLeaderElection.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerZooKeeperServer.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/LeaderZooKeeperServer.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/test/org/apache/zookeeper/server/quorum/StandaloneDisabledTest.java PRE-CREATION
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/ReconfigTest.java 1515597
>   http://svn.apache.org/repos/asf/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/StandaloneTest.java 1515597
>
> Diff: https://reviews.apache.org/r/12983/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Helen Hastings
>
>

+
Edward Ribeiro 2013-08-20, 02:03
+
Edward Ribeiro 2013-08-20, 02:02
+
Edward Ribeiro 2013-08-20, 02:08
+
Edward Ribeiro 2013-08-20, 02:25
+
Edward Ribeiro 2013-08-20, 02:13
+
Edward Ribeiro 2013-08-20, 02:31
+
Helen Hastings 2013-08-21, 22:11
+
Edward Ribeiro 2013-08-22, 00:14
+
Helen Hastings 2013-08-21, 22:11
+
Alexander Shraer 2013-08-30, 11:11
+
Helen Hastings 2013-10-01, 04:02
+
Alexander Shraer 2013-10-03, 04:54
+
Helen Hastings 2013-12-14, 00:00
+
Helen Hastings 2013-12-14, 00:02
+
Helen Hastings 2014-01-07, 04:32
+
Raul Gutierrez Segales 2014-01-17, 18:51