[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-107?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13572665#comment-13572665
 ] 

Alexander Shraer commented on ZOOKEEPER-107:
--------------------------------------------

thanks Ben! the test was removed by mistake - great catch! I'll fix it in the 
next patch I upload.

we did not bump the protocol version - I suggest that if this is needed we do 
that as a separate patch as this may affect other areas in the code besides the 
ones I touched (and may break other things unrelated to reconfiguration :).

LearnerInfo was already changed once without bumping the protocol, so we will 
still have the "if" statement there as we do now.
But if we bump the version we could do what you suggest and get rid of the last 
part of the "if" that I've added.


Alex
                
> Allow dynamic changes to server cluster membership
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-107
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-107
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: server
>            Reporter: Patrick Hunt
>            Assignee: Alexander Shraer
>             Fix For: 3.5.0
>
>         Attachments: SimpleAddition.rtf, zkreconfig-usenixatc-final.pdf, 
> ZOOKEEPER-107-14-Jan.patch, ZOOKEEPER-107-14-Oct.patch, 
> ZOOKEEPER-107-15-Oct.patch, ZOOKEEPER-107-15-Oct-ver1.patch, 
> ZOOKEEPER-107-15-Oct-ver2.patch, ZOOKEEPER-107-15-Oct-ver3.patch, 
> ZOOKEEPER-107-16-Jan.patch, ZOOKEEPER-107-17-Jan.patch, 
> ZOOKEEPER-107-18-Jan.patch, ZOOKEEPER-107-18-Jan-ver2.patch, 
> ZOOKEEPER-107-1-Mar.patch, ZOOKEEPER-107-20-Jan.patch, 
> ZOOKEEPER-107-20-July.patch, ZOOKEEPER-107-21-July.patch, 
> ZOOKEEPER-107-22-Apr.patch, ZOOKEEPER-107-23-SEP.patch, 
> ZOOKEEPER-107-24-Jan.patch, ZOOKEEPER-107-28-Feb.patch, 
> ZOOKEEPER-107-28-Feb.patch, ZOOKEEPER-107-28-NOV-ver2.patch, 
> ZOOKEEPER-107-29-Feb.patch, ZOOKEEPER-107-3-Oct.patch, 
> ZOOKEEPER-107-4-Feb.patch, ZOOKEEPER-107-4-Feb-ver1.patch, 
> ZOOKEEPER-107-4-Feb-ver2.patch, ZOOKEEPER-107-4-Feb-ver2.patch, 
> ZOOKEEPER-107-6-NOV-2.patch, ZOOKEEPER-107-7-NOV.patch, 
> ZOOKEEPER-107-7-NOV-ver1.patch, ZOOKEEPER-107-7-NOV-ver2.patch, 
> ZOOKEEPER-107-Aug-20.patch, ZOOKEEPER-107-Aug-20-ver1.patch, 
> ZOOKEEPER-107-Aug-25.patch, zookeeper-3.4.0.jar, zookeeper-dev-fatjar.jar, 
> zookeeper-reconfig-sep11.patch, zookeeper-reconfig-sep12.patch, 
> zoo_replicated1.cfg, zoo_replicated1.members
>
>
> Currently cluster membership is statically defined, adding/removing hosts 
> to/from the server cluster dynamically needs to be supported.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to