Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23818 )

Change subject: IMPALA-13679: Fix ASAN Failure in Workload Management Tests
......................................................................


Patch Set 5:

(1 comment)

Works for me.

http://gerrit.cloudera.org:8080/#/c/23818/5//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/23818/5//COMMIT_MSG@19
PS5, Line 19: The Ticker class was originally developed to avoid spurious
Was this due to a misunderstanding of what spurious wakeups meant, or just 
overlooked the API? 
https://en.cppreference.com/w/cpp/thread/condition_variable/wait_for.html still 
mentions spurious wakeups.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a29f4f788cf7259c263a17272e86acdda5caa77
Gerrit-Change-Number: 23818
Gerrit-PatchSet: 5
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Yida Wu <[email protected]>
Gerrit-Comment-Date: Tue, 06 Jan 2026 18:46:37 +0000
Gerrit-HasComments: Yes

Reply via email to