> On Aug. 13, 2014, 6:36 a.m., Alexander Shraer wrote: > > src/java/test/org/apache/zookeeper/server/quorum/ReconfigBackupTest.java, > > line 294 > > <https://reviews.apache.org/r/24208/diff/14/?file=658731#file658731line294> > > > > if you take file content, append version=200000, sort, it should be the > > same as what "testServerHasconfig" returns when sorted.
Well, dynamic content are sorted before written to file. But neither testServerHasConfig nor quorumverifier#toString do sorting. So I didn't make such assumptions. - Hongchao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24208/#review50414 ----------------------------------------------------------- On Aug. 12, 2014, 6:54 p.m., Hongchao Deng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24208/ > ----------------------------------------------------------- > > (Updated Aug. 12, 2014, 6:54 p.m.) > > > Review request for zookeeper. > > > Repository: zookeeper-git > > > Description > ------- > > ZOOKEEPER-1994 > > > Diffs > ----- > > src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 76f0afc > src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java > c4397a1 > src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java > 0a8a45a > src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java > 89416ca > src/java/test/org/apache/zookeeper/server/quorum/ReconfigBackupTest.java > PRE-CREATION > src/java/test/org/apache/zookeeper/server/quorum/ReconfigLegacyTest.java > c05aa1b > src/java/test/org/apache/zookeeper/server/quorum/ReconfigRecoveryTest.java > 1a090dc > src/java/test/org/apache/zookeeper/test/LENonTerminateTest.java e9471be > > Diff: https://reviews.apache.org/r/24208/diff/ > > > Testing > ------- > > > Thanks, > > Hongchao Deng > >
