Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9136 )
Change subject: KUDU-1613: evict replicas with wrong server UUIDs ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/9136/3/src/kudu/integration-tests/raft_consensus-itest.cc File src/kudu/integration-tests/raft_consensus-itest.cc: http://gerrit.cloudera.org:8080/#/c/9136/3/src/kudu/integration-tests/raft_consensus-itest.cc@2760 PS3, Line 2760: // time, it means we're not evicting at all. : "--follower_unavailable_considered_failed_sec=10", > Since the 3-4-3 scheme is the default now, does it make sense to adapt this Makes sense, this test was written as a regression test for behavior that happened during the 3-2-3 era, so I went back to the 3-2-3 scheme. You're right though, and the test should still fail without the change in 3-4-3. -- To view, visit http://gerrit.cloudera.org:8080/9136 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If0c7ae9dfbd99c2c9a38e8b1f7748c70882e06b4 Gerrit-Change-Number: 9136 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Thu, 25 Jan 2018 23:31:51 +0000 Gerrit-HasComments: Yes
