ryandotsmith / l2met

Convert a formatted log stream into metrics
http://r.32k.io/l2met-introduction
GNU General Public License v3.0
284 stars 26 forks source link

Be careful with how we use store.Now() #110

Closed ryandotsmith closed 11 years ago

ryandotsmith commented 11 years ago

Currently, the RedisStore relies on a time command sent to Redis. I believe this call is made in a tight loop on bucket parsing. We should probably figure out a more efficient way to get the time from Redis.

danp commented 11 years ago

Looks like you meant to close this, is that right?

ryandotsmith commented 11 years ago

Yes.