Cannot handle times before posix 1000000000

Bugazelle / export-csv-to-influx

The python library to write the CSV data into Influx. Support me via Bitcoin: bc1qqgkmph9cvygzxfpupv4jr4n0nfx3qumwg39j5w

BSD 3-Clause "New" or "Revised" License

35 stars 10 forks source link

Cannot handle times before posix 1000000000 #32

Closed 7yl4r closed 3 years ago

7yl4r commented 3 years ago

When processing 9-digit posix timestamps an error is thrown and the times are not inserted into influxdb. I haven't tested it but I expect all timestamps less than 10 digits long will also fail.

I think the code expects a 10-digit timestamp and appends zeros to the end of the timestamp in an attempt to convert seconds to nanoseconds.

https://github.com/Bugazelle/export-csv-to-influx/blob/eb15d1c6b8932f659459666e60dc1fe9087266cb/src/ExportCsvToInflux/exporter_object.py#L431-L434

Bugazelle commented 3 years ago

Hello @7yl4r

Really apologies for the dealy. Year 2020 is a HELL Mode for me for both life and work.

I will take on this soon these days.

Thank you

Bugazelle commented 3 years ago

Hello @7yl4r :

Really appreciate for the pull requests. You are right, it is a bug.

Thank you for the contribution!! Now your code is in master branch now.

Have a nice day to you.

Cheers, Bugazelle