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.

Sign up for GitHub

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

Jump to bottom

Per-thread vsiopener file registries and exit stacks #2987

Closed
sgillies opened this issue Dec 19, 2023 · 0 comments · Fixed by #2988
Closed

Per-thread vsiopener file registries and exit stacks #2987

sgillies opened this issue Dec 19, 2023 · 0 comments · Fixed by #2988
Assignees
@sgillies
Labels
bug cython
Milestone
1.4.0

Comments

@sgillies
Copy link
Member

It was pointed out in #2898 that the vsiopener state isn't thread-safe. Using contextvars allows us thread local registry of open files and their exit stacks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgillies sgillies

Labels
bug cython
Projects
None yet
Milestone
1.4.0
Development

Successfully merging a pull request may close this issue.

Thread-safe state for visopener and a change note rasterio/rasterio
1 participant
@sgillies