You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Jean-Daniel Cryans (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/01/14 19:48:43 UTC
[kudu-CR] Send back an error when UpdateConsensus cannot prepare a single transaction
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