Skip to content
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

Implementing dropwizard emitter for druid (fixes #3927) #4196

Closed
wants to merge 2 commits into from

Conversation

rohitkochar
Copy link
Contributor

Corresponding issues are
#3927
#1996

@rohitkochar
Copy link
Contributor Author

@b-slim @himanshug
Requesting you to review this pull request.
The code structure is based on the graphite-emitter contributed by @b-slim

@leventov leventov changed the title Implementing dropwizard emitter for druid (https://github.com/druid-io/druid/issues/3927) Implementing dropwizard emitter for druid (fixes #3927) Apr 24, 2017
@jeffwidman
Copy link

jeffwidman commented Nov 10, 2017

Any update on this?

I'm not super familiar with Dropwizard, so I may be conflating things, but I know Kafka uses Dropwizard's metrics library internally, and that's a pretty battle-tested piece of software. So it'd be nice to see this in Druid as well as it'd be very convenient to emit these metrics over JMX/StatsD, etc.

@dkhwangbo
Copy link
Contributor

Hi. Any update? Any schedule to review it?

@jihoonson
Copy link
Contributor

Sorry about the delayed review. @rohitkochar would you please update this PR by merging master? You also need to update the package name and the license of all new files to org.apache.druid and the Apache license, respectively.

@mlubavin-vg
Copy link

This would be great if it would allow emitting metrics in the various dropwizard supported formats. We're specifically looking for prometheus exporting, which can be hooked up through the jmx exporter with https://github.com/prometheus/jmx_exporter

@nishantmonu51
Copy link
Member

@rohitkochar Thanks for the PR. We are also looking for enabling exposing druid metrics via JMX.
@rohitkochar, are you still planning to work on this PR ?

@nishantmonu51
Copy link
Member

FYI, I am working on refactoring this patch for latest druid. Will start by cherry-picking @rohitkochar commit so that he gets credit for the work done.

@nishantmonu51
Copy link
Member

Closing in favor of #7363

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

Successfully merging this pull request may close these issues.

6 participants