Hello Tidy Bot, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16879
to look at the new patch set (#4).
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
---
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, 763 insertions(+), 19 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/16879/4
--
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: newpatchset
Gerrit-Change-Id: Ibb9ba09104761772f9aaffe582776ad34d8dbf57
Gerrit-Change-Number: 16879
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
