You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Andrew Wong (Code Review)" <ge...@cloudera.org> on 2018/01/25 19:34:56 UTC

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

Hello Tidy Bot, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/9136

to look at the new patch set (#2).

Change subject: KUDU-1613: evict replicas with wrong server UUIDs
......................................................................

KUDU-1613: evict replicas with wrong server UUIDs

Currently, if a tablet server is rebuilt before the replicas on it are
evicted, they won't ever be evicted. With this patch, upon receiving a
response with WRONG_SERVER_UUID, leaders will treat the follower as
failed and handle it appropriately.

A new test case is added to raft_consensus-itest that is set up to
prompt eviction (i.e. low timeouts, ample servers to replicated to), and
without the change, it fails.

Change-Id: If0c7ae9dfbd99c2c9a38e8b1f7748c70882e06b4
---
M src/kudu/consensus/consensus_peers.cc
M src/kudu/integration-tests/external_mini_cluster_fs_inspector.cc
M src/kudu/integration-tests/external_mini_cluster_fs_inspector.h
M src/kudu/integration-tests/multidir_cluster-itest.cc
M src/kudu/integration-tests/raft_consensus-itest.cc
M src/kudu/mini-cluster/external_mini_cluster.h
M src/kudu/util/env_util.cc
M src/kudu/util/env_util.h
8 files changed, 117 insertions(+), 32 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/36/9136/2
-- 
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: newpatchset
Gerrit-Change-Id: If0c7ae9dfbd99c2c9a38e8b1f7748c70882e06b4
Gerrit-Change-Number: 9136
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot