Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16569 )

Change subject: KUDU-2612: persist commit MVCC op timestamp in txn metadata
......................................................................


Patch Set 3: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16569/2/src/kudu/tablet/txn_participant-test.cc
File src/kudu/tablet/txn_participant-test.cc:

http://gerrit.cloudera.org:8080/#/c/16569/2/src/kudu/tablet/txn_participant-test.cc@614
PS2, Line 614: // One anchor should be gone and another should be registered in 
its place
             :   // that lasts until we flush the finalized metadata.
             :   ASSERT_EQ(2, 
tablet_replica_->log_anchor_registry()->GetAnchorCountForTests());
> There were previously two anchors. We removed one, and added one, so we're
Ack



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib2400fbb7e96ddba78544a9549965fc095e32ca3
Gerrit-Change-Number: 16569
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 29 Oct 2020 01:18:25 +0000
Gerrit-HasComments: Yes

Reply via email to