Add TCP support
kobynet opened this issue · 8 comments
Statsd server support tcp connection as well as udp. This is sometimes needed especially for accurate counters.
Hi Koby,
Do you have a use case for this now? I know statsd itself supports it, but has always seemed like a shoehorned piece. For accurate counters like this you'd probably want a proper logging platform. If it's going over an unreliable network I'd suggest carbon-relay?
statsd/statsd#318 is a pretty good use case, especially when you do not have permission to change udp buffer size, and in general for sending big chunks (like Batched send etc.).
Is this "Up for grabs"?
@DarrellMozingo I had some free time and played around with it.
You can see the PR #59
Really close to being done with TCP support.
Could use second pair of eyes
Done
@kobynet and @DarrellMozingo
take a look at #59 and leave a comment.