Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Grant Henke, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16710
to look at the new patch set (#5).
Change subject: WIP KUDU-2612 C++ client transaction API
......................................................................
WIP KUDU-2612 C++ client transaction API
This patch adds C++ client API to interact with TxnManager.
Corresponding tests are added as well. End-to-end tests are not
available yet because they require transaction orchestration
functionality to be present, but it's not there yet.
WIP:
* collect feedback on the proposed transaction-related client API
* this patch is rather raw: several TODOs, a few of those might be
addressed even without waiting for the functionality to be
implemented elsewhere
* add more tests
Change-Id: Ic48233f72873012ea36ff4a05d16c58a0ba9b407
---
M src/kudu/client/CMakeLists.txt
M src/kudu/client/client-internal.cc
M src/kudu/client/client-internal.h
M src/kudu/client/client-test.cc
M src/kudu/client/client.cc
M src/kudu/client/client.h
A src/kudu/client/transaction-internal.cc
A src/kudu/client/transaction-internal.h
A src/kudu/client/txn_manager_proxy_rpc.cc
A src/kudu/client/txn_manager_proxy_rpc.h
M src/kudu/common/txn_id.h
M src/kudu/transactions/transactions.proto
12 files changed, 1,134 insertions(+), 24 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/10/16710/5
--
To view, visit http://gerrit.cloudera.org:8080/16710
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic48233f72873012ea36ff4a05d16c58a0ba9b407
Gerrit-Change-Number: 16710
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)