-
Notifications
You must be signed in to change notification settings - Fork 284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ability to submit metrics via named pipes #3230
Conversation
0731ea7
to
ab3cfb1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, apparently CircleCI does have windows support, I think it would be great to get a sense for how many windows users we have, if it's a large enough proportion then we should prioritize getting tests working
@devinsba It was fairly simple to add Windows to the JMXFetch PR: https://github.com/DataDog/java-dogstatsd-client/pull/169/files#diff-78a8a19706dbd2a4425dd72bdab0502ed7a2cef16365ab7030a5a0588927bf47 . The work is really getting the build to work on Windows. |
ab3cfb1
to
2181fbc
Compare
On Windows, allows metrics to be submitted via a named pipe. Most of the changes here are plumbing the named pipe config option through to the dogstatsd client builder.
I also added retry logic to the connection because there is often a race between the start of the dogstatsd server and the tracer starting.
This PR relies on DataDog/java-dogstatsd-client#169 being merged and released (does not compile until it is released)
There are no automated tests in this PR because currently neither the build, nor the testsuite can run on Windows. It would be quite a large effort to get the testsuite working on windows.