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

DM-47889: Prevent DB connection pool exhaustion in Butler server #1124

Merged
merged 3 commits into from
Dec 5, 2024

Limit the maximum retry time for query streaming

8ea635f
Select commit
Loading
Failed to load commit list.
Merged

DM-47889: Prevent DB connection pool exhaustion in Butler server #1124

Limit the maximum retry time for query streaming
8ea635f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 5, 2024 in 1s

92.13% of diff hit (target 89.44%)

View this Pull Request on Codecov

92.13% of diff hit (target 89.44%)

Annotations

Check warning on line 285 in python/lsst/daf/butler/remote_butler/_http_connection.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/remote_butler/_http_connection.py#L284-L285 

Added lines #L284 - L285 were not covered by tests

Check warning on line 211 in python/lsst/daf/butler/remote_butler/server/handlers/_query_streaming.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/remote_butler/server/handlers/_query_streaming.py#L211 

Added line #L211 was not covered by tests

Check warning on line 488 in tests/test_server.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

tests/test_server.py#L488 

Added line #L488 was not covered by tests
View more details on Codecov