Hello Mike Percy, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11836
to look at the new patch set (#2).
Change subject: revert change to exactly_once_writes-itest
......................................................................
revert change to exactly_once_writes-itest
Since bc817a44867c586bf4e0539aa564b282c666a49d, in some environments,
exactly_once_writes-itest will pass successfully but report a leak that
would not fail the test itself.
Try as I might to get to the bottom of the non-fatal leak, its root
cause has evaded me. So instead, I'm reverting the test change that led
to its leakiness. It seems it only fails some of the time when running a
full ctest suite locally (i.e. not through dist-test).
Some conclusions I've drawn about the leak:
* Not easily reproducible via dist-test, even if we begin reporting
LeakSanitizer errors on passing tests (which we don't today)
* Not easily reproducible when running the test locally on its own
* Not deterministically reproducible when running a full ctest suite
locally
Without the change, the leak was detected in 2/7 runs of the full test
suite, and with the change, in 0/14.
Change-Id: If4091d3905d871acb48ec4d88c7b81ee48bf0eed
---
M src/kudu/integration-tests/exactly_once_writes-itest.cc
1 file changed, 2 insertions(+), 13 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/36/11836/2
--
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: newpatchset
Gerrit-Change-Id: If4091d3905d871acb48ec4d88c7b81ee48bf0eed
Gerrit-Change-Number: 11836
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
