Skip to content
This repository has been archived by the owner on Nov 15, 2022. It is now read-only.

Incorrect PRECISION_NANOSECONDS marker #162

Open
bendoh opened this issue Jun 8, 2021 · 0 comments
Open

Incorrect PRECISION_NANOSECONDS marker #162

bendoh opened this issue Jun 8, 2021 · 0 comments

Comments

@bendoh
Copy link

bendoh commented Jun 8, 2021

In https://github.com/influxdata/influxdb-php/blob/master/src/InfluxDB/Database.php#L35

    const PRECISION_NANOSECONDS = 'n';

Technically this is incorrect as the parameter should be 'ns' not 'n', but since the default is nanoseconds anyway, it's basically just a red herring that does nothing. See https://archive.docs.influxdata.com/influxdb/v1.2/tools/api/#write — The "fix", if any, is to just change precision=n to precision=ns, or drop precision altogether and just assume nanoseconds throughout.

@bendoh bendoh changed the title Incorrect PRECISION_NANOSECONDS delimiter Incorrect PRECISION_NANOSECONDS marker Jun 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant