Skip to content
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

Close #47

Open
wants to merge 4 commits into
base: dev/1.0.0
Choose a base branch
from
Open

Close #47

wants to merge 4 commits into from

Conversation

yellowhatter
Copy link

Support background Session close that can be safely waited even in atexit

@yellowhatter yellowhatter self-assigned this Dec 5, 2024
@YuanYuYuan
Copy link
Collaborator

@yellowhatter
Copy link
Author

It really can make finalization things slower (I see it even when running ROS2 examples), but it makes finalization more correct.

@evshary
Copy link

evshary commented Dec 19, 2024

It really can make finalization things slower (I see it even when running ROS2 examples), but it makes finalization more correct.

In fact, the slow we mentioned here is not caused by your PR. This is another bug in Zenoh, and should be fixed by
eclipse-zenoh/zenoh#1649

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants