Skip to content

Changes statsd-emitter error print to a debug print#4111

Merged
fjy merged 1 commit intoapache:masterfrom
Jdban:master
Mar 27, 2017
Merged

Changes statsd-emitter error print to a debug print#4111
fjy merged 1 commit intoapache:masterfrom
Jdban:master

Conversation

@Jdban
Copy link
Copy Markdown
Contributor

@Jdban Jdban commented Mar 24, 2017

This was mentioned in the original pull for the statsd-emitter (#2410) by @sascha-coenen, and the original author (@michaelschiff) agreed that it seemed reasonable

This commit fixes issue #3960

This was mentioned in the original pull (#2410) by @sascha-coenen, and the original author (@michaelschiff) agreed that it seemed reasonable

This commit fixes issue #3960
@fjy
Copy link
Copy Markdown
Contributor

fjy commented Mar 24, 2017

👍

@michaelschiff
Copy link
Copy Markdown
Contributor

@Jdban thanks for doing that, fell off my radar.

Copy link
Copy Markdown
Member

@nishantmonu51 nishantmonu51 left a comment

Choose a reason for hiding this comment

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

👍 , @Jdban Thanks for the contribution, if you haven't already filled out a CLA, could you please do so at: http://druid.io/community/cla.html

@Jdban
Copy link
Copy Markdown
Contributor Author

Jdban commented Mar 27, 2017

@nishantmonu51 - Awesome. I have submitted the CLA.

@fjy fjy added this to the 0.10.1 milestone Mar 27, 2017
@fjy fjy merged commit 117c698 into apache:master Mar 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants