You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2017/11/16 02:58:34 UTC

[kudu-CR] [RaftPeerPB] introduce new status fields

Hello Mike Percy, Kudu Jenkins, Todd Lipcon, 

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

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

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

Change subject: [RaftPeerPB] introduce new status fields
......................................................................

[RaftPeerPB] introduce new status fields

Introduced the following fields for a tablet replica (RaftPeerPB):
  * health_status
  * do_promote
  * do_replace

The 'health_status' is a non-persistent optional field which is set by
leader replica.  It reflects the health of the replica as seen by
the leader.

The 'do_promote' is an optional field which is set by master.  It's
applicable to non-voter replicas only.  If set to 'true', it makes
the leader replica promoting the replica once it catches up with the
leader's WAL.

The 'do_replace' is an optional field which means 'replace the replica
regardless of its health status'.  It's a field to provide the
capability to manually move replica from its current location using the
kudu CLI tool (i.e. 'kudu tablet change_config move_replica').

This commit introduces a single change in the RaftPeerPB protobuf
message.  It provides necessary interface to start working on the new
scheme of automatic replacement of tablet replicas, so it would be
possible to work in parallel in accordance with the v1 implementation
plan.

Change-Id: Id82f838fd3612ab9bc3f91cac7a840cb9f36ff4c
---
M src/kudu/consensus/metadata.proto
1 file changed, 21 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/61/8561/2
-- 
To view, visit http://gerrit.cloudera.org:8080/8561
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id82f838fd3612ab9bc3f91cac7a840cb9f36ff4c
Gerrit-Change-Number: 8561
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>