-
Notifications
You must be signed in to change notification settings - Fork 27
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
Fix issue where Python sources can deadlock when the subscriber is unsubscribed #495
Fix issue where Python sources can deadlock when the subscriber is unsubscribed #495
Conversation
… future and the subscriber to be checked
…nt, revert build_source to using PyIteratorWrapper's range expression
…do not merge [no ci]
…via the CPython api, this avoids an decref error on shutdown
…thread pre-maturely after a copy/move
…1838-python-threaded-queue [no ci]
… constructor will attempt to fetch the first value which could raise an exception
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## branch-24.10 #495 +/- ##
================================================
+ Coverage 74.06% 74.16% +0.10%
================================================
Files 403 403
Lines 14403 14475 +72
Branches 1132 1143 +11
================================================
+ Hits 10667 10736 +69
- Misses 3736 3739 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 8 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Description
python/mrc/_pymrc/src/utilities/acquire_gil.cpp
conftest.py
to set the loglevel toDEBUG
if theGLOG_v
environment variable is defined.Closes nv-morpheus/Morpheus#1838
Checklist