Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17345


Change subject: KUDU-2612: propagate commit timestamp (C++ client)
......................................................................

KUDU-2612: propagate commit timestamp (C++ client)

With this patch, the commit timestamp for a non-empty transaction
is propagated to a Kudu C++ client upon calling
KuduTransaction::IsCommitComplete() (that means that committing
a transaction synchronously via KuduTransaction::Commit() propagates
the commit timestamp as well).

Updating the last observed timestamp with the commit timestamp is
necessary to achieve consistency in the READ_YOUR_WRITES mode when
reading the data of a transaction which has just been committed.  The
commit phase might take some time and may even be retried in some cases,
so even the client observed timestamps for all the write operations
it sent in the context this transaction, the maximum timestamp collected
among the involved transaction participants might be far ahead of the
last timestamp observed by the client so far.

Change-Id: If3ff321895a25326b962a47d5fa868e45aefcb4d
---
M src/kudu/client/transaction-internal.cc
M src/kudu/integration-tests/txn_write_ops-itest.cc
M src/kudu/master/txn_manager.proto
M src/kudu/master/txn_manager_service.cc
M src/kudu/transactions/txn_status_manager.cc
M src/kudu/transactions/txn_system_client.cc
6 files changed, 143 insertions(+), 12 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/17345/1
--
To view, visit http://gerrit.cloudera.org:8080/17345
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If3ff321895a25326b962a47d5fa868e45aefcb4d
Gerrit-Change-Number: 17345
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to