Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/16948
Change subject: WIP [client] updated transaction API
......................................................................
WIP [client] updated transaction API
This patch removed KuduTransactionSerializer and switches to using
KuduTransaction::SerializerOptions instead.
It also introduces KuduTransactionOptions as a placeholder for
future settings for new transactions started by
KuduClient::newTransaction().
WIP:
* Get feedback on the updated API w.r.t. the approach for the
serialization/deserialization and controlling various parameters
of the resulting transaction handle.
* Do we need to introduce the KuduTransactionOptions right now or
we can delay that up to the point where it's really needed?
As for the compatibility perspective, it can be introduced later on
without breaking API/ABI compatibility. At least, that might be
the best approach according to the YAGNI principle.
Change-Id: Id646c41d60f385f079ebfaec2daf7c2a108306b2
---
M src/kudu/client/client-test.cc
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/client_builder-internal.h
M src/kudu/client/transaction-internal.cc
M src/kudu/client/transaction-internal.h
6 files changed, 242 insertions(+), 162 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/16948/1
--
To view, visit http://gerrit.cloudera.org:8080/16948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id646c41d60f385f079ebfaec2daf7c2a108306b2
Gerrit-Change-Number: 16948
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
