[kudu-CR] [consensus] micro-cleanup on VerifyRaftConfig()
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8078 ) Change subject: [consensus] micro-cleanup on VerifyRaftConfig() .. [consensus] micro-cleanup on VerifyRaftConfig() Removed unused RaftConfigState parameter for VerifyRaftConfig(). In addition, fixed typos in metadata.proto inline documentation. Change-Id: Ic5b4cb6ceacde7a0c0b2f1c75dca4b7d1681ef33 Reviewed-on: http://gerrit.cloudera.org:8080/8078 Tested-by: Kudu Jenkins Reviewed-by: Mike Percy --- M src/kudu/consensus/consensus_meta.cc M src/kudu/consensus/metadata.proto M src/kudu/consensus/quorum_util.cc M src/kudu/consensus/quorum_util.h M src/kudu/consensus/raft_consensus.cc M src/kudu/master/sys_catalog.cc 6 files changed, 17 insertions(+), 21 deletions(-) Approvals: Kudu Jenkins: Verified Mike Percy: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8078 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic5b4cb6ceacde7a0c0b2f1c75dca4b7d1681ef33 Gerrit-Change-Number: 8078 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Alexey Serbin Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy
[kudu-CR] [consensus] micro-cleanup on VerifyRaftConfig()
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8078 ) Change subject: [consensus] micro-cleanup on VerifyRaftConfig() .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8078 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic5b4cb6ceacde7a0c0b2f1c75dca4b7d1681ef33 Gerrit-Change-Number: 8078 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy Gerrit-Comment-Date: Tue, 26 Sep 2017 00:51:58 + Gerrit-HasComments: No
[kudu-CR] [consensus] micro-cleanup on VerifyRaftConfig()
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/8078 Change subject: [consensus] micro-cleanup on VerifyRaftConfig() .. [consensus] micro-cleanup on VerifyRaftConfig() Removed unused RaftConfigState parameter for VerifyRaftConfig(). In addition, fixed typos in metadata.proto inline documentation. Change-Id: Ic5b4cb6ceacde7a0c0b2f1c75dca4b7d1681ef33 --- M src/kudu/consensus/consensus_meta.cc M src/kudu/consensus/metadata.proto M src/kudu/consensus/quorum_util.cc M src/kudu/consensus/quorum_util.h M src/kudu/consensus/raft_consensus.cc M src/kudu/master/sys_catalog.cc 6 files changed, 17 insertions(+), 21 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/78/8078/1 -- To view, visit http://gerrit.cloudera.org:8080/8078 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic5b4cb6ceacde7a0c0b2f1c75dca4b7d1681ef33 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin