This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Fix some logging #336

Merged

thuningxu merged 1 commit into master from logs

Nov 30, 2017

Contributor

thuningxu commented Nov 30, 2017

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

thuningxu requested a review from datoug

November 30, 2017 00:47


          Fix some logging

thuningxu force-pushed the logs branch from 6732980 to 4126767 Compare

November 30, 2017 00:49

datoug approved these changes

View reviewed changes

coveralls commented Nov 30, 2017 •

edited

Loading

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

thuningxu merged commit d24994e into master

thuningxu deleted the logs branch

November 30, 2017 03:00

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet