Skip to content
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

Refactor tutorial_udp example for timestamps #808

Merged
merged 1 commit into from
Apr 10, 2020

Conversation

shantanoo-desai
Copy link
Contributor

@shantanoo-desai shantanoo-desai commented Apr 8, 2020

  • time key should be within each datapoint

  • Align the Points

  • Insert distinct timestamps to observe difference in insertion

  • This PR addresses make_lines does not provide timestamps in the resultant line protocol string  #788 regarding the structure of the data when
    inserting via UDP.

  • The original documentation contributed by me that took the structure of
    the tutorial.py as base. However, upon testing, the timestamp in the
    example are not written (2020 is written as opposed to 2009).

  • Tested for influxdb-python v5.2.3 and InfluxDB v1.6.1


Contributor checklist
  • Builds are passing
  • New tests have been added (for feature additions)

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
- `time` key should be within each datapoint

- This PR addresses influxdata#788 regarding the structure of the data when
inserting via UDP.
- The original documentation contributed by me took the structure of
the `tutorial.py` as base. However, upon testing, the timestamp in the
example are not written (2020 is written as opposed to 2009).

- Tested for `influxdb-python` v5.2.3 and InfluxDB v1.6.1
@sebito91 sebito91 self-assigned this Apr 8, 2020
Copy link
Contributor

@sebito91 sebito91 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix @shantanoo-desai !

@sebito91 sebito91 merged commit f7f30b5 into influxdata:master Apr 10, 2020
@shantanoo-desai
Copy link
Contributor Author

closes #788

@shantanoo-desai shantanoo-desai deleted the UDP branch April 10, 2020 18:31
ocworld pushed a commit to AhnLab-OSS/influxdb-python that referenced this pull request Apr 13, 2020
- `time` key should be within each datapoint

- This PR addresses influxdata#788 regarding the structure of the data when
inserting via UDP.
- The original documentation contributed by me took the structure of
the `tutorial.py` as base. However, upon testing, the timestamp in the
example are not written (2020 is written as opposed to 2009).

- Tested for `influxdb-python` v5.2.3 and InfluxDB v1.6.1
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.

None yet

2 participants