contribsys / faktory

Language-agnostic persistent background job server
https://contribsys.com/faktory/
Other
5.78k stars 230 forks source link

[Feature Request] Include Redis RTT in statsd metrics that are emitted #475

Closed yvjessestephens closed 6 months ago

yvjessestephens commented 6 months ago

Some Redis metrics are already available (see Ent-Metrics):

I'm interested in being able to capture the RTT so that we can alert if that metric exceeds its baseline by very much and investigate what might have changed to cause that (e.g. the SaaS Redis instance got moved to a different zone), and to track that metric over time for historical performance.

yvjessestephens commented 6 months ago

@mperham , now that you have marked this as completed, in what version could I expect this feature to be available?

mperham commented 6 months ago

I don't have an ETA on 1.9.0, I was trying to land #474 first but it's blocked until I have a way to reproduce and verify the PR improves things. I might need to adjust my release plan.

mperham commented 6 months ago

this is now available in 1.9