Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

filter: New "failed_nats_publish" metric #68

Merged
merged 1 commit into from
Apr 24, 2018

Conversation

mjs
Copy link
Contributor

@mjs mjs commented Apr 24, 2018

Previously, errors while publishing to NATS were being silently ignored. This now increments the new metric and generates logs if Debug is enabled.

Previously, errors while publishing to NATS were being silently
ignored. This now increments the new metric and generates logs if
Debug is enabled.
@mjs mjs requested a review from oplehto April 24, 2018 04:28
Copy link
Contributor

@oplehto oplehto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@oplehto oplehto merged commit f836098 into jumptrading:master Apr 24, 2018
@mjs mjs deleted the filter-publish-errors branch April 24, 2018 04:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants