From ae206fc243f9c0a477318d88301fbf8318f4c7a2 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Wed, 18 Sep 2019 16:25:29 -0600 Subject: [PATCH 1/3] added a note to line protocol doc recommending including timestamps, resolves #466 --- content/v2.0/reference/line-protocol.md | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/content/v2.0/reference/line-protocol.md b/content/v2.0/reference/line-protocol.md index 8d89d9ecb..0498bd217 100644 --- a/content/v2.0/reference/line-protocol.md +++ b/content/v2.0/reference/line-protocol.md @@ -93,7 +93,12 @@ The Unix nanosecond timestamp for the data point. InfluxDB accepts one timestamp per point. If no timestamp is provided, InfluxDB uses the system time (UTC) of its host machine. -_**Data type:** [Unix timestamp](#unix-timestamp)_ +_**Data type:** [Unix timestamp](#unix-timestamp)_ + +{{% note %}} +Though timestamps are optional, you should include them to ensure times associated with points +are the actual times metrics were observed rather than when InfluxDB received them. +{{% /note %}} {{% note %}} _Use the default nanosecond precision timestamp or specify an alternative precision From 06de76528fe3bcef85d287dfc2fbf9f00b6b459b Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Wed, 18 Sep 2019 16:27:02 -0600 Subject: [PATCH 2/3] minor update to line protocol doc --- content/v2.0/reference/line-protocol.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/content/v2.0/reference/line-protocol.md b/content/v2.0/reference/line-protocol.md index 0498bd217..276a2aee6 100644 --- a/content/v2.0/reference/line-protocol.md +++ b/content/v2.0/reference/line-protocol.md @@ -96,7 +96,7 @@ If no timestamp is provided, InfluxDB uses the system time (UTC) of its host mac _**Data type:** [Unix timestamp](#unix-timestamp)_ {{% note %}} -Though timestamps are optional, you should include them to ensure times associated with points +Though timestamps are optional, include them to ensure times associated with points are the actual times metrics were observed rather than when InfluxDB received them. {{% /note %}} From 32185c2a266424f739d03815a75bd8f443663985 Mon Sep 17 00:00:00 2001 From: Scott Anderson Date: Wed, 18 Sep 2019 18:43:56 -0600 Subject: [PATCH 3/3] updated line protocol doc to address PR feedback --- content/v2.0/reference/line-protocol.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/content/v2.0/reference/line-protocol.md b/content/v2.0/reference/line-protocol.md index 276a2aee6..5d43cfa63 100644 --- a/content/v2.0/reference/line-protocol.md +++ b/content/v2.0/reference/line-protocol.md @@ -96,8 +96,8 @@ If no timestamp is provided, InfluxDB uses the system time (UTC) of its host mac _**Data type:** [Unix timestamp](#unix-timestamp)_ {{% note %}} -Though timestamps are optional, include them to ensure times associated with points -are the actual times metrics were observed rather than when InfluxDB received them. +To ensure a data point includes the time a metric is observed (not received by InfluxDB), +include the timestamp. {{% /note %}} {{% note %}}