search

organicveggie / metrics-statsd

Statsd reporter for codahale/metrics.
Apache License 2.0
93 stars 80 forks source link

Limit the size of a single datagram and send multiple datagrams if you h... #7

Open matthewbogner opened 11 years ago

matthewbogner commented 11 years ago

...ave more metrics than can fit into a single UDP packet. Without limiting the size of the datagram, it's possible your packet will fail to be sent.

Review on Reviewable

gitlakerzhou commented 10 years ago

@matthewbogner I am not sure if I have the same issue as you described. I can only see some of my metrics sending to StatsD from wireshark. I don't see any error message. Majority of my metrics seems silently dropped. To me, version 2.3 is not usable.

JensRantil commented 10 years ago

This is a very serious bug. I agree with previous commenter that it makes 2.3.0 practically unusable.

JensRantil commented 10 years ago

15 should fix this. Just need a test. Feel free to contribute to it.