Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9781 )
Change subject: consensus: Expose health status in GetConsensusState() RPC ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/9781/1/src/kudu/consensus/consensus.proto File src/kudu/consensus/consensus.proto: http://gerrit.cloudera.org:8080/#/c/9781/1/src/kudu/consensus/consensus.proto@479 PS1, Line 479: UNSPECIFIED_HEALTH_REPORT = 0; : EXCLUDE_HEALTH_REPORT = 1; : INCLUDE_HEALTH_REPORT = 2; > This is a namespace-level enum so that would create consensus::UNKNOWN, etc Did you see it's conflicted when you tried it? http://gerrit.cloudera.org:8080/#/c/9781/1/src/kudu/consensus/consensus.proto@495 PS1, Line 495: IncludeHealthReport > Can you think of a scenario where we will need that? If not, it seems reaso That sounds reasonable as well, especially if you don't expect this to change ever. -- To view, visit http://gerrit.cloudera.org:8080/9781 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I15b8931156183e1713b902d22abbc80ca393f6b9 Gerrit-Change-Number: 9781 Gerrit-PatchSet: 1 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Comment-Date: Fri, 23 Mar 2018 23:05:46 +0000 Gerrit-HasComments: Yes
