Skip to content
This repository has been archived by the owner on Jun 12, 2020. It is now read-only.

Add TCP support #51

Closed
kobynet opened this issue Oct 13, 2015 · 8 comments
Closed

Add TCP support #51

kobynet opened this issue Oct 13, 2015 · 8 comments

Comments

@kobynet
Copy link

kobynet commented Oct 13, 2015

Statsd server support tcp connection as well as udp. This is sometimes needed especially for accurate counters.

@DarrellMozingo
Copy link
Collaborator

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?

@kobynet
Copy link
Author

kobynet commented Oct 28, 2015

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

@pekiZG
Copy link
Contributor

pekiZG commented Apr 13, 2016

Is this "Up for grabs"?

@pekiZG
Copy link
Contributor

pekiZG commented Apr 13, 2016

@DarrellMozingo I had some free time and played around with it.
You can see the PR #59

@pekiZG
Copy link
Contributor

pekiZG commented Apr 19, 2016

Really close to being done with TCP support.
Could use second pair of eyes

@pekiZG
Copy link
Contributor

pekiZG commented Apr 25, 2016

Done
@kobynet and @DarrellMozingo
take a look at #59 and leave a comment.

@pekiZG
Copy link
Contributor

pekiZG commented Apr 28, 2016

from 1.3.43 version TCP client is available.

@DarrellMozingo
Copy link
Collaborator

Closed in #59 & live in 1.3.44. Thanks @pekiZG!

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

No branches or pull requests

3 participants