-
Notifications
You must be signed in to change notification settings - Fork 49
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
Use logstash redis defaults. #25
Use logstash redis defaults. #25
Conversation
|
if_p('redis.congestion_threshold') do |threshold| | ||
output['options']['congestion_threshold'] = threshold | ||
end | ||
%> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
72 to 87 are the changes introduced in this PR. Most of the rest is whitespace.
0076df1
to
a86bf56
Compare
a86bf56
to
5582293
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @jmcarp, everything looks fine, but could you please update redis-input-plugin for parser logstash instances to also use the ELK default settings.
f92f7f0
to
f5f5ed6
Compare
The only non-default option I saw for redis input was |
LGTM |
[Resolves #24]
Most of the diffs here are changing tabs to spaces. The only meaningful change is dropping the default redis congestion properties, then only adding them to the redis options if defined.