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

Kairosdb ttl support #2252

Merged
merged 5 commits into from Apr 18, 2017
Merged

Kairosdb ttl support #2252

merged 5 commits into from Apr 18, 2017

Conversation

jaroug
Copy link
Contributor

@jaroug jaroug commented Apr 14, 2017

Had a new option for the kairosdb ouput format.

TTL let you sets the Cassandra ttl for the data points.

@@ -1547,6 +1547,7 @@
# SSLVersion "TLSv1"
# Format "Command"
# Attribute "key" "value" # only available for KAIROSDB format
# TTL 0 # data ttl, only available for KAIRODB format
Copy link
Contributor

Choose a reason for hiding this comment

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

KAIROSDB

@@ -8778,6 +8778,14 @@ each metric being sent out.

You can add multiple B<Attribute>.

=item B<TTL> I<Int>

Only available for KAIROSDB output format.
Copy link
Contributor

Choose a reason for hiding this comment

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

the KAIROSDB output format

Copy link
Contributor

Choose a reason for hiding this comment

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

Please fix this one too, then I'll merge.

Copy link
Contributor

Choose a reason for hiding this comment

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

No you didn't. you fixed the comment after "Attribute", but not after "TTL"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just saw my mistake, my bad :)

Copy link
Contributor

Choose a reason for hiding this comment

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

no problem :)

@rubenk rubenk merged commit 16bf49d into collectd:master Apr 18, 2017
@octo octo added this to the 5.8 milestone Oct 11, 2017
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.

None yet

3 participants