Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16710 )
Change subject: KUDU-2612 C++ client transaction API ...................................................................... KUDU-2612 C++ client transaction API This patch adds C++ client API to interact with TxnManager. Corresponding tests are added as well. True end-to-end tests are not available yet because they require transaction orchestration functionality to be present, but it's not there yet. I'm planning to add more tests as soon as the orchestration starts to appear. This patch is mostly focused on the client-side transactional API, containing several TODOs. The scope of those is limited to the related client-side-only functionality. I'm planning to address those in follow-up patches. I think that way it's easier to scope, implemented and review compared with lumping them all in a single patch. Change-Id: Ic48233f72873012ea36ff4a05d16c58a0ba9b407 Reviewed-on: http://gerrit.cloudera.org:8080/16710 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- 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,431 insertions(+), 24 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- 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: merged Gerrit-Change-Id: Ic48233f72873012ea36ff4a05d16c58a0ba9b407 Gerrit-Change-Number: 16710 Gerrit-PatchSet: 10 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)
