search

fangli / fluent-plugin-influxdb

A buffered output plugin for fluentd and InfluxDB
MIT License
111 stars 65 forks source link

Set timestamp with js new Date().getTime() ms will meet timestamp format error #85

Open ArthurJiang opened 6 years ago

ArthurJiang commented 6 years ago

config: <match .influxdb.test.health> @type influxdb measurement: health host 127.0.0.1 port 40002 dbname test auto_tags true tag_keys: ["key1", "key2"] time_key time time_precision ms

plugin will auto convert js "new Date().getTime()" integer value to a float, I guess it's an issue for ruby integer scope, I convert "new Date().getTime()" to a string as a work around.

comments1: I think all timestamp must be an integer, so could you add an auto integer convert for timestamp.

ash0080 commented 6 years ago

Same issue, Is this project still maintained?

repeatedly commented 6 years ago

plugin will auto convert js "new Date().getTime()" integer value to a float, I guess it's an issue for ruby integer scope, I convert "new Date().getTime()" to a string as a work around.

Ruby's integer range is unlimited because Ruby's integer uses BigInteger for big number. What does 'js' mean? Do you use fluentd javascript client for sending logs to fluentd?

ash0080 commented 6 years ago

For me, I am using fluent-logger-node now