Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/17233 )
Change subject: KUDU-2612: allow ABORT_TXN txn ops to succeed if txn hasn't started ...................................................................... Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/17233/4/src/kudu/tablet/txn_participant.h File src/kudu/tablet/txn_participant.h: http://gerrit.cloudera.org:8080/#/c/17233/4/src/kudu/tablet/txn_participant.h@268 PS4, Line 268: If we're initializing a new transaction, checks for any persiste > Now whis became confusing: what if 'state_' == kCommitted and expected_meta Good call, I'll rewrite this to clarify a bit. http://gerrit.cloudera.org:8080/#/c/17233/4/src/kudu/tablet/txn_participant.h@290 PS4, Line 290: return Status::IllegalState( > readability nit: would it be easier to reason about possible outcomes if it Done -- To view, visit http://gerrit.cloudera.org:8080/17233 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia4c864b4f14e42008d3aa8f4454c8b2abf9bb766 Gerrit-Change-Number: 17233 Gerrit-PatchSet: 5 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 03 Apr 2021 02:18:36 +0000 Gerrit-HasComments: Yes
