Hello Hao Hao,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/10770
to review the following change.
Change subject: KUDU-2480: tsan failure of master-stress-test
......................................................................
KUDU-2480: tsan failure of master-stress-test
This is an attempt to fix the race reported in KUDU-2480. I've been
unable to reproduce the race on dist-test, however after auditing the
code mentioned the the stack traces I saw this bit of fishy code. My
current theory is that it's illegal to call
Synchronizer::AsStatusCallback twice on one synchronizer, which was
accidentally happening due to some dead code. This may or may not fix
the race, but it's a mistake worth fixing nonetheless.
Change-Id: I6b5103242aa24ffa81813d8e6ab5df630c69c8cf
---
M src/kudu/master/hms_notification_log_listener.cc
1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/10770/1
--
To view, visit http://gerrit.cloudera.org:8080/10770
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6b5103242aa24ffa81813d8e6ab5df630c69c8cf
Gerrit-Change-Number: 10770
Gerrit-PatchSet: 1
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
