Skip to content

grass.pygrass: Fix resource_tracker semaphore warning at exit - #7833

Merged
petrasovaa merged 1 commit into
OSGeo:mainfrom
saket0187:fix-temporal-semaphore-warning
Aug 21, 2026
Merged

petrasovaa merged 1 commit into
OSGeo:mainfrom
saket0187:fix-temporal-semaphore-warning

Conversation

@saket0187

Copy link
Copy Markdown
Contributor

Closes #7832

Unlink the RPC server lock semaphore when the server is stopped, so temporal tools no longer emit a resource_tracker warning at exit.

Note :- Messenger holds the same kind of lock, but bisecting showed only the CLibrariesInterface lock survives to shutdown, so changing its stop() would fix nothing.

@petrasovaa

@github-actions github-actions Bot added Python Related code is in Python libraries labels Aug 21, 2026

@echoix echoix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without testing, it looks in the good direction to me. I’m happy to see someone else inspect the inner workings of that specific area of the code. I had such a hard time profiling that, and I’m convinced there’s something else wrong with the mechanism that degrades the performance so much.

@petrasovaa
petrasovaa merged commit 8f39c47 into OSGeo:main Aug 21, 2026
26 checks passed
@github-actions github-actions Bot added this to the 8.6.0 milestone Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libraries Python Related code is in Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] t.* tools emit a "leaked semaphore" warning at exit on Python 3.14

3 participants