Skip to content
This repository was archived by the owner on Nov 25, 2020. It is now read-only.

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

Open
matthewbogner wants to merge 1 commit into
organicveggie:masterfrom
lithiumtech:multiple-datagrams
Open

Limit the size of a single datagram and send multiple datagrams if you h...#7
matthewbogner wants to merge 1 commit into
organicveggie:masterfrom
lithiumtech:multiple-datagrams

Conversation

@matthewbogner

Copy link
Copy Markdown

...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

…u have more metrics than can fit into a single UDP packet
@gitlakerzhou

Copy link
Copy Markdown

@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

Copy link
Copy Markdown

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

@JensRantil JensRantil mentioned this pull request Jun 3, 2014
@JensRantil

Copy link
Copy Markdown

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants