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/28 05:45:20 UTC

[kudu-CR] [consensus queue] update replica health reporting

Hello Mike Percy, Kudu Jenkins, 

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

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

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

Change subject: [consensus_queue] update replica health reporting
......................................................................

[consensus_queue] update replica health reporting

Report the health status of a replica as HEALTHY only if the
last recent exchage status was Status::OK.  Except for a few
other definitive cases where it's clear that the replica has failed,
report the health of the replica is reported as UNKNOWN.

The rationalte behind is simple:

  * The HEALTHY replicas are the ones which can replicate Raft
    transactions from the leader replica with no issues.

  * The FAILED replicas are the ones which will definitely fail
    to replicate Raft transactions from the leader replica.

  * The rest is goes into the UNKNOWN basket, which covers the cases
    when there hasn't been a single attempt to contact the replica yet,
    there was a transient error during the last communication with the
    replica, etc.

Change-Id: I99da4352da0b0d1846c92eb2abac7197a58bfd62
---
M src/kudu/consensus/consensus_queue.cc
1 file changed, 17 insertions(+), 15 deletions(-)


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

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