Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16729
to look at the new patch set (#11).
Change subject: KUDU-2612 keep-alive tracking for transactions
......................................................................
KUDU-2612 keep-alive tracking for transactions
This patch introduces the functionality of tracking the liveness of
a distributed multi-row transaction into TxnStatusManager and
provides corresponding proxy methods in TxnManager, so a Kudu client
now can send keep-alive requests for a transaction (the implementation
of the latter is planned in a follow-up patch).
>From the TxnStatusManager, the newly introduced keep-alive RPC is
represented as another type of CoordinateTransaction() request:
CoordinatorOpPB::KEEP_TXN_ALIVE. New tests to cover the existing
functionality are added as well.
More end-to-end tests will be added by follow-up changelist once Kudu
C++ client starts sending keepalive requests for started transactions.
Also, all newly introduced tests in txn_status_manager-itest.cc are
disabled because without [1] they are a bit flaky. I added TODO
to re-enable those once [1] is committed.
[1] https://gerrit.cloudera.org/#/c/16648/
Change-Id: Iae926e02fa7ca597b63ccea90124964c3b6a1175
---
M src/kudu/integration-tests/CMakeLists.txt
M src/kudu/integration-tests/ts_tablet_manager-itest.cc
A 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/master/txn_manager.cc
M src/kudu/tablet/txn_coordinator.h
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
M src/kudu/transactions/txn_status_tablet.h
M src/kudu/transactions/txn_system_client.cc
M src/kudu/transactions/txn_system_client.h
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/ts_tablet_manager.cc
M src/kudu/tserver/ts_tablet_manager.h
18 files changed, 1,034 insertions(+), 59 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/29/16729/11
--
To view, visit http://gerrit.cloudera.org:8080/16729
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iae926e02fa7ca597b63ccea90124964c3b6a1175
Gerrit-Change-Number: 16729
Gerrit-PatchSet: 11
Gerrit-Owner: Alexey Serbin <aser...@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)
