On 20 June 2015 at 17:07, Michi Mutsuzaki mutsuz...@gmail.com wrote:
I updated https://issues.apache.org/jira/browse/ZOOKEEPER-2210 to
address feedback from Chris. Could somebody take a look? Thanks!
Merged - thanks!
-rgs
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35204/#review88715
---
src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35204/#review88716
---
src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14595276#comment-14595276
]
Raul Gutierrez Segales commented on ZOOKEEPER-2193:
---
One last nit,
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14595272#comment-14595272
]
Raul Gutierrez Segales commented on ZOOKEEPER-2193:
---
Patch lgtm, +1.
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14595160#comment-14595160
]
Chris Nauroth commented on ZOOKEEPER-2210:
--
+1 (non-binding). I retested