Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17022 )
Change subject: KUDU-2612: allow aborting after beginning to commit ...................................................................... KUDU-2612: allow aborting after beginning to commit This patch adjusts the TxnStatusManager state machine to include a new FINALIZE_IN_PROGRESS state to serve as an intermediate step between COMMIT_IN_PROGRESS and COMMITTED. The goal is to allow for aborts to occur in the event that we've begun committing, but not yet called FINALIZE_COMMIT on any of the transaction's participants, which may be desirable in cases where anything surprising has happened on the participants (e.g. if they're deleted). Change-Id: If1b6596df2db5601f7e17e528ad6dc68057b67f8 Reviewed-on: http://gerrit.cloudera.org:8080/17022 Tested-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java M src/kudu/client/transaction-internal.cc M src/kudu/integration-tests/txn_commit-itest.cc M src/kudu/master/txn_manager-test.cc M src/kudu/tablet/txn_participant-test.cc M src/kudu/transactions/transactions.proto M src/kudu/transactions/txn_status_entry.cc M src/kudu/transactions/txn_status_entry.h M src/kudu/transactions/txn_status_manager-test.cc M src/kudu/transactions/txn_status_manager.cc M src/kudu/transactions/txn_status_manager.h 11 files changed, 690 insertions(+), 175 deletions(-) Approvals: Andrew Wong: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17022 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If1b6596df2db5601f7e17e528ad6dc68057b67f8 Gerrit-Change-Number: 17022 Gerrit-PatchSet: 11 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
