> On Sept. 27, 2016, 6:37 p.m., Rakesh R wrote: > > src/java/main/org/apache/zookeeper/KeeperException.java, line 317 > > <https://reviews.apache.org/r/51546/diff/3/?file=1510450#file1510450line317> > > > > Do we need @Deprecated constant value? Since we are in 3.5 branch, I > > feel its not required. > > > > Secondly, we can't we use -122? > > Michael Han wrote: > Thanks for all the reviews, Rakesh! > > We need deprecated here but I see what you mean - instead of adding a > deprecated value I can just skip adding it. > > -122 is used in zookeeper.h as ZRWSERVERFOUND... this maybe a bug as > there is no corresponding code in Java client. I'd like to deal with this > later so skip -122 and choose next value -123. > > Rakesh R wrote: > How about raising a tracking jira to revisit this part. Thanks!
Just for the record, rased ZOOKEEPER-2627 to track this part. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51546/#review150538 ----------------------------------------------------------- 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 > >