Skip to content

Expose useful pool stats - #24

Open
wrl314 wants to merge 6 commits into
bos:masterfrom
wrl314:master
Open

Expose useful pool stats#24
wrl314 wants to merge 6 commits into
bos:masterfrom
wrl314:master

Conversation

@wrl314

Copy link
Copy Markdown

Various stats about pool activity exposed via 'stats' function. Useful when tied into a monitoring framework to determine if pool is properly sized.

Stats:

  • highwater usage
  • current usage
  • number of takes
  • number of creates
  • number of creation failures

@pheaver

Copy link
Copy Markdown

Hello, can someone comment on whether this PR may be accepted or not? We have been using our patched version of this library in production for a while now and would love to switch to the official one.

@domenkozar

Copy link
Copy Markdown

@bos this seems useful to us as well, is there something missing for merge?

@chreekat

Copy link
Copy Markdown

I recall bos may be easier to reach by email than by github notification, @pheaver and @domenkozar.

@avanovavanov mentioned this pull request May 23, 2021
@avanov

Copy link
Copy Markdown

avanov pushed a commit to avanov/pool that referenced this pull request Dec 14, 2021
@avanovavanov mentioned this pull request Dec 14, 2021
Sign up for freeto 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.

6 participants

@wrl314@pheaver@domenkozar@chreekat@avanov@cydparser