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

Add support for time precision when using influxdb as output #5645

Conversation

StephenSorriaux
Copy link

This allows to set the time precision (s, ms, us, ns) when sending metrics to InfluxDB. Previously, only ns was possible.

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

This allows to set the time precision (s, ms, us, ns) when sending metrics to InfluxDB. Previously, only ns was possible.
@StephenSorriaux StephenSorriaux force-pushed the feat/add-time-precision-influxdb-output branch from 8f55843 to d636976 Compare March 28, 2019 10:12
@@ -48,6 +48,7 @@ type InfluxDB struct {
SkipDatabaseCreation bool `toml:"skip_database_creation"`
InfluxUintSupport bool `toml:"influx_uint_support"`
tls.ClientConfig
TimePrecision string `toml:"time_precision"`

Precision string // precision deprecated in 1.0; value is ignored
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure the reason, but I believe this functionality was deprecated intentionally.

@danielnelson
Copy link
Contributor

You can use the agent precision to round values instead. We actually used to have this functionality but it was removed #1368 because it was no longer needed.

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.

3 participants