Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/1785
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 a UpdateConsensus() call fails to prepare a single, 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/raft_consensus.cc M src/kudu/integration-tests/raft_consensus-itest.cc 2 files changed, 69 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/1785/1 -- To view, visit http://gerrit.cloudera.org:8080/1785 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I546fd3069af974383c23acb7406ea621e6962bb3 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans
