Skip to content
This repository was archived by the owner on Oct 5, 2022. It is now read-only.

Conversation

QijingLi
Copy link

No description provided.

Copy link

@jeffwidman jeffwidman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some additional context, since I'm also involved with this project internally where @QijingLi works:

Druid is an open-source columnar datastore that is used by a number of companies for fast analytical queries.

The Druid project currently ships with a StatsD emitter. We modified that code to emit DogStatsD with tags.
Right now, it's a bit of a hack, so we are working on cleaning it up to submit upstream to Druid.

However, in parallel with this, we wanted to start the review process with the Datadog team for whitelisting these metrics. So can you please review and let us know what you think?

I'll be at Pycon this week (planning to attend the Datadog APM tutorial tomorrow), so perhaps I can review this list of metrics in-person with someone on the team?

CC @irabinovitch as he and I discussed this project at the Datadog customer summit in SF.

<li>Configure Druid(
<a href="http://druid.io/docs/latest/development/extensions-contrib/statsd.html">see doc<a/>)<br>
<pre class="textfile">
<code> Exampel Configuration in common.runtime.properties:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exampel --> Example

@irabinovitch
Copy link
Contributor

@jeffwidman Thanks for this. A few notes:

  • The Druid logo looks a bit off. Could we get a version with the correct aspect ratio for each size file?
  • We would want to hold for the emitter to be merged upstream before we add this tile and docs n our end.

@irabinovitch
Copy link
Contributor

@jeffwidman Is there a PR we can follow upstream?

@jeffwidman
Copy link

Afraid not, as @QijingLi is currently on paternity leave. That's why I haven't pushed this at all. We'll pick it up when he returns.

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