Todd Lipcon has submitted this change and it was merged.

Change subject: tsan: remove two old suppressions
......................................................................


tsan: remove two old suppressions

This removes two TSAN suppressions which have been in the list since the
very first TSAN commit. I went back and looked at that code review and
can't find any explanation for them, and initial local testing with them
removed doesn't seem to turn anything up.

Maybe we've since addressed these and just didn't remove them. Let's
remove them, and if we start to see any TSAN flakes due to them, we can
add them back with an appropriate comment explaining why they're
required.

Change-Id: I25354ce37300ba495aa73909b6a2ba3672f22906
Reviewed-on: http://gerrit.cloudera.org:8080/3411
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <a...@cloudera.com>
---
M build-support/tsan-suppressions.txt
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3411
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I25354ce37300ba495aa73909b6a2ba3672f22906
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to