Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12114 )
Change subject: [tests] add /threadz page to PeriodicWebUIChecker ...................................................................... Patch Set 1: > Does this cause your next patch to trigger a TSAN data race in > linked_list-test without that locking fix? Thank you for fast review. This didn't cause TSAN data race at the place where I expected it since the race would be exposed for /threads endpoint, not /threads?group=all (I realized it after posting this patch). I now updating the test to make sure TSAN shows some races. -- To view, visit http://gerrit.cloudera.org:8080/12114 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iad6cbfe5108ac99231b82f662cee56ffcd25f026 Gerrit-Change-Number: 12114 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 20 Dec 2018 01:53:15 +0000 Gerrit-HasComments: No
