TODO: Configurable enable/disable of last_msg_seen #375

Closed
draco2003 opened this Issue Dec 6, 2013 · 3 comments

Projects

None yet

2 participants

@draco2003

If you aren't using this metric, than it is a lot of overhead per packet received and disabling it for a busy statsd process has performance gains.

https://github.com/etsy/statsd/blob/master/stats.js#L246

Would be great to add a configuration flag to enable/disable this

@mrtazz
Etsy, Inc. member

How much would we gain in terms of performance? Isn't that very negligible based on the amount of all the other metrics we send?

@draco2003
@mrtazz
Etsy, Inc. member

I don't think we are gaining that much from this. I think calculating a timestamp once per packet is probably less work than everything else we do.

@mrtazz mrtazz closed this Jan 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment