-
Notifications
You must be signed in to change notification settings - Fork 439
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
More plentiful "snapshot" value in circuit breaker observer functions #183
Comments
Hi, we've already carried previous state, triggered rule and "snapshot" value (if needed) in the observer functions. You may refer to the example: https://github.com/alibaba/sentinel-golang/blob/master/example/circuitbreaker/circuit_breaker_example.go |
Thanks very much the cur state that I mean is not the open state or half open state, I mean the cur metrics, such as cur request number or cur rt. because I want to do some alert with cur info when system is breaking on the other hand,I have reviewed the code in core/log/metric , because we have our own monitor system,I suggest the metric need easier to access to other systems |
Currently only the triggered ratio was provided. We may improve the "snapshot" value later so that it could provide more useful metrics.
We're working on it. We may discuss it in #70 |
Thanks |
Issue Description
Type: feature request
Describe what feature you want
Snapshot is interface,I think it is for more plentiful data,but we only pass a simple param when we use it,I think we can pass more information,such as cur state,and we can get more information in the listener.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: