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