Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14816 )
Change subject: mvcc: some locking cleanup ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14816/1/src/kudu/tablet/mvcc.cc File src/kudu/tablet/mvcc.cc: http://gerrit.cloudera.org:8080/#/c/14816/1/src/kudu/tablet/mvcc.cc@230 PS1, Line 230: (*iter)->latch->CountDown(); > What if 'iter' returned at line 229 is waiters_.end()? Good catch. I'll fix this. -- To view, visit http://gerrit.cloudera.org:8080/14816 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I189553d0f589794ad830a30b3c0b4ce5fd5569ba Gerrit-Change-Number: 14816 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 03 Dec 2019 21:03:38 +0000 Gerrit-HasComments: Yes
