search

RedisTimeSeries / prometheus-redistimeseries-adapter

Prometheus remote storage adapter for RedisTimeSeries
https://redistimeseries.io
BSD 3-Clause "New" or "Revised" License
33 stars 11 forks source link

fix #40: all timestamps should be in millis #41

Closed danni-m closed 5 years ago

RoeyPrat commented 5 years ago

https://github.com/RedisTimeSeries/prometheus-redistimeseries-adapter/blob/560f908463fa6bbf0da92beaaaaceb3eb7b4861d/internal/redis_ts/client_test.go#L59 I think the unit tests still assume seconds

gkorland commented 5 years ago

@danni-m can we extend the tests to cover it for next time?