Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

lower "fitting was gone" log to debug level #75

Merged
merged 1 commit into from May 21, 2013

Conversation

Projects
None yet
2 participants
Contributor

beerriot commented May 2, 2013

Issue #50 is not the only complaint about the message,
"Pipe worker startup failed:fitting was gone before startup".
It's not an important message, though. It might tell you that a
pipe vnode is lagging a bit, but most often it's just noise about
pipes that shut down for other reasons (MapReduce errors).

This commit lowers the log level of the message to 'debug' and
rewords it, since it really has nothing to do with the worker
failing to start.

Bryan Fink lower "fitting was gone" log to debug level
Issue #50 is not the only complaint about the message,
"Pipe worker startup failed:fitting was gone before startup".
It's not an important message, though. It *might* tell you that a
pipe vnode is lagging a bit, but most often it's just noise about
pipes that shut down for other reasons (MapReduce errors).

This commit lowers the log level of the message to 'debug' and
rewords it, since it really has nothing to do with the worker
failing to start.
ca225ff

@beerriot beerriot was assigned May 2, 2013

Contributor

metadave commented May 21, 2013

+1

@beerriot beerriot pushed a commit that referenced this pull request May 21, 2013

Bryan Fink Merge pull request #75 from branch 'bwf-lower-log-level' 8730e98

@beerriot beerriot merged commit ca225ff into master May 21, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment