Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
ArthurJiang opened this issue Feb 8, 2018 · 3 comments

Comments

@ArthurJiang
Copy link

ArthurJiang commented Feb 8, 2018

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
Copy link

ash0080 commented Aug 26, 2018

Same issue, Is this project still maintained?

@repeatedly
Copy link
Collaborator

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
Copy link

ash0080 commented Aug 30, 2018

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants