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

Gauge - there is livesum and all but what about sum? #793

Closed
nkov opened this issue Apr 6, 2022 · 4 comments
Closed

Gauge - there is livesum and all but what about sum? #793

nkov opened this issue Apr 6, 2022 · 4 comments

Comments

@nkov
Copy link

nkov commented Apr 6, 2022

I have a multi-process application where multiple processes are used to track a single value. For instance, lets say I have 3 processes: A, B, and C behind a load balancer.

A receives a request to increment the Gauge: A = 1, B = 0, C = 0
B receives a request to decrement the Gauge: A = 1, B = -1, C = 0

So using livesum, my current gauge value is 0 (as expected). However, what would happen if B suddenly needs to terminate (eg. due to auto-scaling) or restart (due to a crash, memory management, etc.)? How does the client handle this? Do I assume correctly that the Gauge will now show the incorrect value of 1, because "live sum" only considers the living processes?

@csmarchbanks
Copy link
Member

Your analysis is correct, #794 was just opened to add the missing modes, so hopefully there will be a sum option soon!

@csmarchbanks
Copy link
Member

Fixed in #794

@nkov
Copy link
Author

nkov commented May 21, 2022

@csmarchbanks Awesome, what a quick turnaround. When could we expect this to be released to PyPI?

@csmarchbanks
Copy link
Member

I usually try to cut a release about once every three months but there isn't a specific cadence. I may try to review the multiprocess help text PR and cut a release after that is merged though.

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

No branches or pull requests

2 participants