Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9255 )
Change subject: KUDU-2274 [itest] stress test for replica replacement ...................................................................... Patch Set 5: Verified+1 (1 comment) Unreleated flakes in DiskReservationITest.TestFillMultipleDisks DiskReservationITest.TestWalWriteToFullDiskAborts due to NTP failure F0214 06:53:26.686384 28842 master_main.cc:74] Check failed: _s.ok() Bad status: Service unavailable: Cannot initialize clock: Error reading clock. Clock considered unsynchronized http://gerrit.cloudera.org:8080/#/c/9255/3/src/kudu/integration-tests/raft_consensus_stress-itest.cc File src/kudu/integration-tests/raft_consensus_stress-itest.cc: http://gerrit.cloudera.org:8080/#/c/9255/3/src/kudu/integration-tests/raft_consensus_stress-itest.cc@113 PS3, Line 113: > Since we are not using tablet_id_ etc, can we instead use ExternalMiniClust This test uses tablet_servers_ and the code to populate that. I'm also thinking that we might expand functionality of this test in the future, so I would prefer to keep RaftConsensusITestBase here. However, if you feel strongly about that, I'll update this to use ExternalMiniClusterITestBase. -- To view, visit http://gerrit.cloudera.org:8080/9255 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I036a882f7e9132a5c26013227c50c0699b59ed6e Gerrit-Change-Number: 9255 Gerrit-PatchSet: 5 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Wed, 14 Feb 2018 19:49:02 +0000 Gerrit-HasComments: Yes
