Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11885 )
Change subject: Limit number of rowsets in compaction selection to 32 in TSAN mode ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/11885/1/src/kudu/tablet/tablet.cc File src/kudu/tablet/tablet.cc: http://gerrit.cloudera.org:8080/#/c/11885/1/src/kudu/tablet/tablet.cc@1378 PS1, Line 1378: take its compact_flush_lock > See L1407 below. The lock is moved into the 'picked' collection. Right, it seems I missed that the move() part. http://gerrit.cloudera.org:8080/#/c/11885/1/src/kudu/tablet/tablet.cc@1385 PS1, Line 1385: #if defined(THREAD_SANITIZER) > This is an alternative way to do indentation? I'm not understanding what th Yep; the point was to keep the '#' in the beginning of the line. It might be easier to read, but I don't see that in the google style guide, so feel free to ignore. -- To view, visit http://gerrit.cloudera.org:8080/11885 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I01ad4ba3a13995c194c3308d72c1eb9b611ef766 Gerrit-Change-Number: 11885 Gerrit-PatchSet: 1 Gerrit-Owner: Will Berkeley <[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-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Tue, 06 Nov 2018 06:33:09 +0000 Gerrit-HasComments: Yes
