logstash-plugins / logstash-output-influxdb

Apache License 2.0
58 stars 77 forks source link

remove to_epoc #4

Closed jsvd closed 9 years ago

jsvd commented 9 years ago

to_epoc converts "event.timestamp" to a unix timestamp, but if the argument is not a Time object it firsts tries Time.parse(arg)

However, event.timestamp should always return a LogStash::Timestamp object, so the method will generate an exception since passing a Logstash::Timestamp to Time.parse coerces it to a string that Time.parse can't handle.

Fixes #3

ph commented 9 years ago

LGTM

elasticsearch-bot commented 9 years ago

Merged sucessfully into master!