Jean-Daniel Cryans has uploaded a new patch set (#3). Change subject: Send back an error when UpdateConsensus cannot prepare a single transaction ......................................................................
Send back an error when UpdateConsensus cannot prepare a single transaction If for any reason an UpdateConsensus() call fails to prepare a single op, it will still return OK. The leader doesn't detect that no progress was made, so it sends a new batch right away... and it's the same one. This patch makes it so that we detect this situation and so that we go into the error-handling path on response. It also adds a test where we manufacture the same conditions. Change-Id: I546fd3069af974383c23acb7406ea621e6962bb3 --- M src/kudu/consensus/consensus.proto M src/kudu/consensus/raft_consensus.cc M src/kudu/integration-tests/raft_consensus-itest.cc 3 files changed, 70 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/1785/3 -- To view, visit http://gerrit.cloudera.org:8080/1785 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I546fd3069af974383c23acb7406ea621e6962bb3 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
