You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Andrew Wong (Code Review)" <ge...@cloudera.org> on 2021/03/09 21:30:24 UTC

[kudu-CR] KUDU-2612: allow aborting after beginning to commit

Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Hao Hao, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/17022

to look at the new patch set (#4).

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_TXN 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
---
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, 627 insertions(+), 172 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/17022/4
-- 
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: newpatchset
Gerrit-Change-Id: If1b6596df2db5601f7e17e528ad6dc68057b67f8
Gerrit-Change-Number: 17022
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)