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