elastic / elasticsearch-metrics-reporter-java

Metrics reporter, which reports to elasticsearch
Apache License 2.0
230 stars 103 forks source link

Using millisecond precision (instead of second) for @timestamp #27

Closed glureau closed 2 years ago

glureau commented 8 years ago

Working on a project where we need a report every second, the current precision is giving us sometimes two events with the same timestamp. From what I see in the project, we are divising and multiplying by 1000 to have variables in seconds but at the end exporting a Date that has the ms precision. I think it should be preferable to use directly the ms everywhere but maybe I missed something here?

(note: tried on my project, all reports are good on ELK so I can prepare a PR quickly, but I want to be sure it's not just a personal need. Thoughts?)

glureau commented 2 years ago

Closing old issues.