Alexey Serbin has submitted this change and it was merged. ( 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 Reviewed-on: http://gerrit.cloudera.org:8080/17345 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- 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, 142 insertions(+), 12 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Alexey Serbin: Verified -- 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: merged Gerrit-Change-Id: If3ff321895a25326b962a47d5fa868e45aefcb4d Gerrit-Change-Number: 17345 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
