search

uber-archive / cherami-server

Distributed, scalable, durable, and highly available message queue system. This project is deprecated and not maintained.
https://eng.uber.com/cherami/
MIT License
1.42k stars 102 forks source link

Fix some logging #336

Closed thuningxu closed 6 years ago

thuningxu commented 6 years ago
  1. messagecache.go: the code intended to rate limit logging to be once per minute, but it forgets to save the last logging timestamp.
  2. outputhost.go: OpenConsumerStreamHandler() need not re-log whatever error in OpenConsumerStream() that is already logged.
coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.07%) to 66.642% when pulling 4126767af8b067b007ae5ff762638a70d92a0966 on logs into 02eb433e61b04b3813f8cac845bd92d54428de73 on master.