Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8561 )
Change subject: [RaftPeerPB] introduce replica health status ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8561/1/src/kudu/consensus/metadata.proto File src/kudu/consensus/metadata.proto: http://gerrit.cloudera.org:8080/#/c/8561/1/src/kudu/consensus/metadata.proto@75 PS1, Line 75: This is run-time : // only field, it should not be persisted or read from the persistent storage. is this true even on the master? or would we expect the master to persist it? if the master doesn't persist it somehow, and the master fails over, how does the new master not get super confused about what's going on with the "extra" replicas? -- 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: comment Gerrit-Change-Id: Id82f838fd3612ab9bc3f91cac7a840cb9f36ff4c Gerrit-Change-Number: 8561 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Thu, 16 Nov 2017 01:30:32 +0000 Gerrit-HasComments: Yes