Andrew Wong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17368 )

Change subject: KUDU-2612: re-open TxnSystemTable if transaction is in new range
......................................................................

KUDU-2612: re-open TxnSystemTable if transaction is in new range

This patch makes the TxnSystemClient on tablet servers retry if attempts
to register the participant yield a NotFound error. This happens when
new ranges are added to the transactions status table.

To disambiguate between the case where the transaction itself doesn't
exist on the TxnStatusManager, this patch also adjust the return value
for such cases to be InvalidArugment, which seems equally fitting for
writing as a part of a bogus transaction ID.

Change-Id: I3af58dcaa3a995dac9dc937c7dfcb652bf004873
Reviewed-on: http://gerrit.cloudera.org:8080/17368
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
---
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTransaction.java
M src/kudu/integration-tests/txn_status_table-itest.cc
M src/kudu/integration-tests/txn_write_ops-itest.cc
M src/kudu/master/txn_manager-test.cc
M src/kudu/transactions/txn_status_manager-test.cc
M src/kudu/transactions/txn_status_manager.cc
M src/kudu/tserver/ts_tablet_manager.cc
7 files changed, 58 insertions(+), 23 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17368
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3af58dcaa3a995dac9dc937c7dfcb652bf004873
Gerrit-Change-Number: 17368
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to