Use absl::Notification when it makes sense#21446
Use absl::Notification when it makes sense#21446rgs1 wants to merge 1 commit intoenvoyproxy:mainfrom
Conversation
I had to run some code in the main thread for a test recently and the first example I bumped into wasn't using absl::Notification, so let's port them over. Signed-off-by: Raul Gutierrez Segales <rgs@pinterest.com>
|
cc: @JuniorHsu |
|
This is a follow-up to #16946. cc: @antoniovicente |
|
/retest |
|
Retrying Azure Pipelines: |
|
CI failures seem legit: |
|
This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
github-actions
bot
added
the
stale
|
This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
3 participants
I had to run some code in the main thread for a test recently
and the first example I bumped into wasn't using absl::Notification,
so let's port them over.
Signed-off-by: Raul Gutierrez Segales rgs@pinterest.com