puma-plugin-statsd

puma-plugin-statsd copied to clipboard

We are running some apps on an EKS IPv6 cluster and puma-plugin-statsd errored out on our deployments, so this PR contains a fix. Inspired from https://github.com/skroutz/logstash-input-gelf/pull/1/files

chen-anders

Backport crash fix to v1.x release

1

Is it possible to backport the fix provided in #35 to the v1.x release, so that users still on puma 4.x can benefit from it?

ctrombley

Improve performance using `Array#sum`

2

The following micro benchmark shows `Array#sum` is better than `Array#map` with `Array#inject`. ``` ruby require 'benchmark' array = 4.times.to_a Benchmark.bm(24) {|x| x.report('Array#sum') do 100000.times { array.sum(0, &:itself) } end x.report('Array#map...

tricknotes

I am getting the below error very frequently after enabling this plugin, used datadog socket in container to push the metrics to datadog ! statsd: notify stats failed: #

knidhin1

Add some further clarifications to the gauge descriptions

3

I’ve linked some references below (from puma 5 since it's the min supported version). If the reasoning or accuracy of these changes is still uncertain, please feel free to let...

joshuay03

andrewle

Negative requests count metric on server shutdown

1

We're seeing this plugin report a negative requests count metric just before Puma (in our case in a Kubernetes pod) shuts down. I'm assuming that the counters are cleared at...

tjvc