Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16879 )
Change subject: KUDU-2612: add RPC to send participant ops ...................................................................... KUDU-2612: add RPC to send participant ops This adds methods to the TxnSystemClient to send participant ops to participants by their tablet ID. This will be used in steps 13 and 18 of the transactions write path[1]. The new ParticipantRpc abstraction borrows a lot from CoordinatorRpc with regards to lookups and error handling, with the following differences: - Rather than doing the lookup by table and partition key, it performs a lookup by tablet ID, using the functionality recently added to the MetaCache. - Since TxnParticipants don't return success on repeated participant op requests calls, some additional handling is done for the TXN_OP_ALREADY_APPLIED error code. [1] https://docs.google.com/document/d/1qv7Zejpfzg-HvF5azRL49g5lRLQ4437EmJ53GiupcWQ/edit#heading=h.4lm41o75ev1x Change-Id: Ibb9ba09104761772f9aaffe582776ad34d8dbf57 Reviewed-on: http://gerrit.cloudera.org:8080/16879 Tested-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/client/client.h M src/kudu/client/meta_cache.cc M src/kudu/integration-tests/txn_participant-itest.cc M src/kudu/tablet/txn_participant-test-util.h M src/kudu/transactions/CMakeLists.txt A src/kudu/transactions/participant_rpc.cc A src/kudu/transactions/participant_rpc.h M src/kudu/transactions/txn_system_client.cc M src/kudu/transactions/txn_system_client.h 9 files changed, 806 insertions(+), 36 deletions(-) Approvals: Andrew Wong: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16879 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ibb9ba09104761772f9aaffe582776ad34d8dbf57 Gerrit-Change-Number: 16879 Gerrit-PatchSet: 6 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
