-
Notifications
You must be signed in to change notification settings - Fork 524
Nanosecond time data being sent even if time_precision set to something else #221
Comments
Don't worry, your feedback is appreciated :) Actually, the time_precision parameter is not meant to convert your time precision. So I guess that's a feature request? |
Hmm, that might be handy if it's not a feature already :) It looked like the time parameter was getting converted though, i.e. I was specifying a timestamp down to microsecond precision with:
which was then sent as:
So it looks like it added '104' as nanoseconds from somewhere, instead of sending the value as '1438001748165231', which was the data specified in the 'time' field as a string. |
@dschallis , just to confirm, the new feature you'd like is to specify a given |
@ChristopherRabotin I think it was more the other way around, i.e. if I specify |
@aviau , do you reckon this should be a feature to be supported in this client? I think it should be supported at the InfluxDB level directly, especially to avoid having different clients behaving in different ways. |
I was under the impression that most users sent integers anyways. The way its implemented in #224 is not intrusive as you can just send integers and it won't be touched. |
Just to clarify a little. Implementation in #224 also handles |
In 2.7.1, if I have e.g.:
Then the 'precision' field is correctly set, but the timestamp is sent as a nanosecond integer value, e.g. the above is sent with:
This leads to it being silently rejected by InfluxDB (i.e. after the above, a
SELECT * FROM foo
returns 0 results), which is probably a bug on their side too (though they don't document very well what should happen in this case).(apologies for the many tickets by the way, I'm in the process of switching to this module from a reasonably sized code base, so starting to use this module quite heavily :) )
The text was updated successfully, but these errors were encountered: