Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11836 )
Change subject: revert change to exactly_once_writes-itest ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/11836/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/11836/2//COMMIT_MSG@20 PS2, Line 20: which we don't today > Leaks don't cause test failures? When did that change? I wonder if it's whe It seems in some cases, no. I'm still trying to get to the bottom of this, but what I think I'm seeing is that in the EMC process complains about a leak throughout the test, and at the very end when doing the leak check, the leak is somehow now caught. In many, if not most cases, the leak will be caught by the leak check that happens in the shutdown of ExternalDaemon. -- To view, visit http://gerrit.cloudera.org:8080/11836 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If4091d3905d871acb48ec4d88c7b81ee48bf0eed Gerrit-Change-Number: 11836 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Comment-Date: Thu, 01 Nov 2018 22:12:26 +0000 Gerrit-HasComments: Yes