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

feat(nsq_consumer): Add input plugin to consume metrics from an nsqd topic #1369

Closed
wants to merge 1 commit into from

Conversation

jchauncey
Copy link

@jchauncey jchauncey commented Jun 13, 2016

This plugin allows a user to configure telegraf to pull metric data from an NSQD instance. The data must be in the line protocol format.

@jchauncey jchauncey force-pushed the nsq-consumer branch 2 times, most recently from a5cb014 to 6162030 Compare June 13, 2016 17:17
}

// SetParser takes the data_format from the config and finds the right parser for that format
func (nsqConsumer *NSQConsumer) SetParser(parser parsers.Parser) {
Copy link
Contributor

Choose a reason for hiding this comment

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

it's better go style to give this a shorter name internally, like

func (n *NSQConsumer) SetParser(parser parsers.Parser)

}

var sampleConfig = `
## An array of NSQD HTTP API endpoints
Copy link
Contributor

Choose a reason for hiding this comment

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

it's not an array, just a string. Should it be an array?

@sparrc
Copy link
Contributor

sparrc commented Jun 22, 2016

this will close #1347

@sparrc
Copy link
Contributor

sparrc commented Jun 23, 2016

thanks @jchauncey

chebrolus pushed a commit to chebrolus/telegraf that referenced this pull request Jun 24, 2016
to consume metrics from an nsqd topic

closes influxdata#1347
closes influxdata#1369
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.

2 participants