-
Notifications
You must be signed in to change notification settings - Fork 61
[#139] fix event concurrency issues #186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#139] fix event concurrency issues #186
Conversation
… goes out of scope
…oblems does not cause failure on notifier side
… used concurrently
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. The commented out code could be removed, either in this PR or a follow up PR.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #186 +/- ##
==========================================
+ Coverage 78.54% 78.62% +0.08%
==========================================
Files 181 181
Lines 19837 19922 +85
==========================================
+ Hits 15580 15663 +83
- Misses 4257 4259 +2
|
Notes for Reviewer
Pre-Review Checklist for the PR Author
SPDX-License-Identifier: Apache-2.0 OR MIT
iox2-123-introduce-posix-ipc-example
)[#123] Add posix ipc example
)task-list-completed
)Checklist for the PR Reviewer
Post-review Checklist for the PR Author
References
Closes #139