Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16992 )
Change subject: KUDU-2612: loosen restrictions for BEGIN_COMMIT ops ...................................................................... Patch Set 2: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/16992/1/src/kudu/integration-tests/txn_participant-itest.cc File src/kudu/integration-tests/txn_participant-itest.cc: http://gerrit.cloudera.org:8080/#/c/16992/1/src/kudu/integration-tests/txn_participant-itest.cc@568 PS1, Line 568: TabletServerErrorPB::Code code = TabletServerErrorPB::UNKNOWN_ERROR; : ASSERT_OK(ParticipateInTransactionCheckResp( : admin_proxy.get(), tablet_id, kTxnId, ParticipantOpPB::BEGIN_COMMIT, : &code, &begin_commit_ts)); : ASSERT_EQ(TabletServerErrorPB::UNKNOWN_ERROR, code); : ASSERT_NE(Timestamp::kInvalidTimestamp, begin_commit_ts); > Right, this is tested at L645. Cool: thanks for the pointer! http://gerrit.cloudera.org:8080/#/c/16992/1/src/kudu/tablet/txn_participant.h File src/kudu/tablet/txn_participant.h: http://gerrit.cloudera.org:8080/#/c/16992/1/src/kudu/tablet/txn_participant.h@135 PS1, Line 135: ValidateBeginCommit > Yeah it's intentional; the Status::OK case is the common case. Whoops, indeed: there 'state_ != kOpen' conditino in the very end of this method (probably, I misinterpreted it). Thanks for confirming. -- To view, visit http://gerrit.cloudera.org:8080/16992 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifa4c5314190c84648c1b1edea7aab776b4882f97 Gerrit-Change-Number: 16992 Gerrit-PatchSet: 2 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-Comment-Date: Sun, 31 Jan 2021 17:35:31 +0000 Gerrit-HasComments: Yes
