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

Feat/deadman #982

Closed
wants to merge 7 commits into from
Closed

Feat/deadman #982

wants to merge 7 commits into from

Conversation

jrconlin
Copy link
Member

Description

Ops:

Adds SYNC_DATABASE_POOL_CONNECTION_DEADMAN_SWITCH which is the
number of milliseconds the pool can report being at 0 available
connections before the application triggers a panic!.

The current default is 0 meaning the deadman switch is inactive.

Testing

TBD

Issue(s)

Issue #945

@jrconlin jrconlin requested a review from a team January 26, 2021 22:17
jrconlin and others added 6 commits January 26, 2021 17:17
*Ops:*

Adds `SYNC_DATABASE_POOL_CONNECTION_DEADMAN_SWITCH` which is the
number of milliseconds the pool can report being at 0 available
connections before the application triggers a `panic!`.

The current default is `0` meaning the deadman switch is inactive.

Issue #945
Closes #984
From @56quarters in #981

Delegate to count method and use glob import for metric traits

(resubmitting due to github commit signature issues -- jrc)
@jrconlin
Copy link
Member Author

jrconlin commented Feb 1, 2021

Closing in favor of #985

@jrconlin jrconlin closed this Feb 1, 2021
@jrconlin jrconlin deleted the feat/deadman branch February 23, 2021 18:15
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.

Add deadman switch if a connection pool stays exhausted for a period of time
1 participant