> On Oct. 13, 2016, 9:57 p.m., Abraham Fine wrote: > > src/docs/src/documentation/content/xdocs/zookeeperAdmin.xml, line 1134 > > <https://reviews.apache.org/r/51546/diff/10/?file=1531330#file1531330line1134> > > > > is there a way for us to sanity check this (make sure that their > > individual reconfigEnabled matches that of the leader) when servers join > > the quorum? > > Michael Han wrote: > I am not aware of any approach of doing the check - probably document it > like this is enough. Looking forward hearing what others think about this.
Thanks Abraham for bringing out this point. I'm OK with the clear documentation and the present explanation about the "inconsistent behabior" looks good to me. - Rakesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51546/#review152577 ----------------------------------------------------------- On Oct. 31, 2016, 4:31 a.m., Michael Han wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51546/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2016, 4:31 a.m.) > > > Review request for zookeeper, fpj, Patrick Hunt, Raul Gutierrez Segales, and > Alexander Shraer. > > > Bugs: ZOOKEEPER-2014 > https://issues.apache.org/jira/browse/ZOOKEEPER-2014 > > > Repository: zookeeper-git > > > Description > ------- > > Address various security concerns around reconfig feature (ZOOKEEPER-2014) to > unblock 3.5.3 release. > > > Diffs > ----- > > build.xml 5328035 > src/c/include/zookeeper.h 18a203d > src/c/tests/TestReconfigServer.cc 6a429ac > src/c/tests/ZooKeeperQuorumServer.h aa8b7cc > src/c/tests/ZooKeeperQuorumServer.cc 23392cd > src/docs/src/documentation/content/xdocs/zookeeperAdmin.xml 5a30da8 > src/docs/src/documentation/content/xdocs/zookeeperReconfig.xml 7168a01 > src/java/main/org/apache/zookeeper/ClientCnxn.java 12dd51c > src/java/main/org/apache/zookeeper/KeeperException.java a05f1ab > src/java/main/org/apache/zookeeper/ZooKeeper.java 1c9ed4e > src/java/main/org/apache/zookeeper/ZooKeeperMain.java c39395a > src/java/main/org/apache/zookeeper/admin/ZooKeeperAdmin.java PRE-CREATION > src/java/main/org/apache/zookeeper/cli/CliCommand.java 3d0a90b > src/java/main/org/apache/zookeeper/cli/ReconfigCommand.java deb7914 > src/java/main/org/apache/zookeeper/server/DataTree.java 4666578 > src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java 7a72757 > src/java/main/org/apache/zookeeper/server/NettyServerCnxnFactory.java > 25b682b > src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java e275f9c > src/java/main/org/apache/zookeeper/server/ServerCnxnFactory.java fc6766c > src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java f94c54d > src/java/main/org/apache/zookeeper/server/ZKDatabase.java 05bbb91 > src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java d4f150b > src/java/main/org/apache/zookeeper/server/ZooKeeperServerMain.java 885a5e1 > src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 68bef42 > src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java > e772fa8 > src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java > bfe8588 > src/java/systest/org/apache/zookeeper/test/system/BaseSysTest.java 109c1b5 > src/java/test/org/apache/zookeeper/TestableZooKeeper.java 4d46fdf > src/java/test/org/apache/zookeeper/server/CRCTest.java da4ebaf > src/java/test/org/apache/zookeeper/server/DataTreeTest.java d726643 > src/java/test/org/apache/zookeeper/server/SessionTrackerTest.java 87aa73e > src/java/test/org/apache/zookeeper/server/ZooKeeperServerMainTest.java > 1030209 > src/java/test/org/apache/zookeeper/server/quorum/CommitProcessorTest.java > c6ee230 > src/java/test/org/apache/zookeeper/server/quorum/FLELostMessageTest.java > cc44243 > src/java/test/org/apache/zookeeper/server/quorum/LearnerHandlerTest.java > 95e5e53 > src/java/test/org/apache/zookeeper/server/quorum/LearnerTest.java 4debe74 > src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTest.java > fae7e5b > src/java/test/org/apache/zookeeper/server/quorum/RaceConditionTest.java > ef7f3df > src/java/test/org/apache/zookeeper/server/quorum/ReconfigBackupTest.java > 241af52 > > src/java/test/org/apache/zookeeper/server/quorum/ReconfigDuringLeaderSyncTest.java > 301837d > > src/java/test/org/apache/zookeeper/server/quorum/ReconfigFailureCasesTest.java > e7147b3 > src/java/test/org/apache/zookeeper/server/quorum/ReconfigLegacyTest.java > ee9f2e2 > > src/java/test/org/apache/zookeeper/server/quorum/StandaloneDisabledTest.java > 1f6ce1f > src/java/test/org/apache/zookeeper/server/quorum/Zab1_0Test.java 6a01447 > src/java/test/org/apache/zookeeper/test/ACLTest.java 9920fc4 > src/java/test/org/apache/zookeeper/test/ClientBase.java 309b2b2 > src/java/test/org/apache/zookeeper/test/FLEPredicateTest.java a4244d8 > src/java/test/org/apache/zookeeper/test/LENonTerminateTest.java 2bbf7b5 > src/java/test/org/apache/zookeeper/test/QuorumBase.java f687f45 > src/java/test/org/apache/zookeeper/test/QuorumUtil.java 314171d > src/java/test/org/apache/zookeeper/test/QuorumUtilTest.java 76e6df0 > src/java/test/org/apache/zookeeper/test/ReconfigExceptionTest.java > PRE-CREATION > src/java/test/org/apache/zookeeper/test/ReconfigMisconfigTest.java > PRE-CREATION > src/java/test/org/apache/zookeeper/test/ReconfigTest.java 248a754 > src/java/test/org/apache/zookeeper/test/StandaloneTest.java 5c95280 > > Diff: https://reviews.apache.org/r/51546/diff/ > > > Testing > ------- > > > Thanks, > > Michael Han > >
