Hello Kudu Jenkins, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17073
to look at the new patch set (#2).
Change subject: KUDU-2612: make BeginCommit return OK if already committed
......................................................................
KUDU-2612: make BeginCommit return OK if already committed
This patch makes repeat calls to BeginCommit by the TxnStatusManager
return OK rather than an IllegalState error. This has previously
surfaced in flakiness in some tests, wherein repeated calls to
TxnManager::Commit() yields an IllegalState error since the commit has
already completed.
This also reverts 67018be8ba27480b050c11504df8a732f6a52daf that
addressed this by increasing commit latency.
Change-Id: I420ba1c5460d59103984ea9a1f16177b82b8d0e1
---
M src/kudu/integration-tests/txn_commit-itest.cc
M src/kudu/transactions/txn_status_manager-test.cc
M src/kudu/transactions/txn_status_manager.cc
3 files changed, 4 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/17073/2
--
To view, visit http://gerrit.cloudera.org:8080/17073
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I420ba1c5460d59103984ea9a1f16177b82b8d0e1
Gerrit-Change-Number: 17073
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
