Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address WSA_IO_PENDING warnings in log files #14

Closed
MikeChristensen opened this issue Apr 10, 2018 · 3 comments
Closed

Address WSA_IO_PENDING warnings in log files #14

MikeChristensen opened this issue Apr 10, 2018 · 3 comments

Comments

@MikeChristensen
Copy link

This is in reference to this issue, which seems to be specific to the Windows port of Redis. We get about 2 of these warnings logged every second for each machine in our cluster. There doesn't appear to be a way to turn them off without turning off all of logging. This is causing an issue both due to hard disk space, as well as costing us money since we're using Splunk to index log files. I'd love to see a way to suppress these warnings, or a work around for the issue.

@tporadowski
Copy link
Owner

Yes, it is specific to Windows port. It's logged with severity WARNING and indeed cannot be avoided without turning logging off completely. It is logged in src/cluster.c: https://github.com/tporadowski/redis/blob/win-4.0.2/src/cluster.c#L2118-L2119

I am not sure about the purpose of this information, but we could:

  1. decrease its severity (to verbose or debug)
  2. remove it completely.

@tporadowski
Copy link
Owner

Ouch, now I know what you mean... I'm trying to reproduce an issue in cluster mode (#12) and it is indeed annoying and not really that much helpful - I will set it to use debug verbosity.

tporadowski added a commit that referenced this issue Apr 11, 2018
- clusterWriteDone() and clusterWriteHandler() are now using LL_DEBUG verbosity instead of LL_WARNING to inform about not-so-mportant events

Solves #14
@tporadowski
Copy link
Owner

Resolved in 6262460

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants