Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17017 )
Change subject: KUDU-2612: add background task to abort transaction participants ...................................................................... KUDU-2612: add background task to abort transaction participants This patch implements background tasks that abort a given transaction when TxnStatusManager::AbortTransaction() is called. Similar to the commit tasks, aborts have the following life cycle: 1. AbortTransaction() is called. A new state, ABORT_IN_PROGRESS, is written to the TxnStatusManager. 2. ABORT_TXN ops are sent to all participants in the transaction. 3. Once all participants have responded, the ABORTED state is written to the TxnStatusManager. This patch doesn't test races between commits and aborts. Some reworking of the commit tasks will be required to account for such races, and will be done in a follow-up. Change-Id: I484c315c6f7331c5ec12cb06370fbaae9c7c343e Reviewed-on: http://gerrit.cloudera.org:8080/17017 Tested-by: Andrew Wong <aw...@cloudera.com> Reviewed-by: Hao Hao <hao....@cloudera.com> --- M java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTransaction.java M src/kudu/client/client-test.cc M src/kudu/client/transaction-internal.cc M src/kudu/integration-tests/txn_commit-itest.cc M src/kudu/integration-tests/txn_status_manager-itest.cc M src/kudu/integration-tests/txn_status_table-itest.cc M src/kudu/master/txn_manager-test.cc M src/kudu/transactions/transactions.proto 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 12 files changed, 422 insertions(+), 61 deletions(-) Approvals: Andrew Wong: Verified Hao Hao: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17017 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I484c315c6f7331c5ec12cb06370fbaae9c7c343e Gerrit-Change-Number: 17017 Gerrit-PatchSet: 9 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
