Skip to content
This repository has been archived by the owner on Nov 14, 2019. It is now read-only.

Support backend of opentsdb #36

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

ccl0326
Copy link

@ccl0326 ccl0326 commented Feb 18, 2016

as title.

@@ -13,6 +13,7 @@ all: default
SOURCES = \
src/backend.c \
src/backends/carbon.c \
src/backends/opentsdb.c \

Choose a reason for hiding this comment

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

Not sure if github is deceiving me, but seems to be like you have a tabs vs spaces issue here and below in the enum.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed.^^

@moises-silva
Copy link

+1

@liyichao
Copy link

You can support opentsdb's tag by allowing application to encode it via key, see https://github.com/danslimmon/statsd-opentsdb-backend#configuration

@vmg
Copy link
Contributor

vmg commented Feb 23, 2016

Looks pretty good. Let me backport this to our internal fork and I'll push it back. 👌

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.

4 participants