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
Add workers() option to the Redis destination #3732
Conversation
This user does not have permission to start the build. Can one of the admins verify this patch and start the build? |
1 similar comment
This user does not have permission to start the build. Can one of the admins verify this patch and start the build? |
@kira-syslogng ok to test |
Build SUCCESS |
Build SUCCESS |
a13c5a9
to
a887890
Compare
@kira-syslogng add to whitelist |
Build SUCCESS |
Build FAILURE |
Build SUCCESS |
Build SUCCESS |
Build SUCCESS |
2ea7bde
to
5568746
Compare
Build SUCCESS |
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.
Perfect, thanks!
#3741 has been merged. |
Signed-off-by: Parrag Szilárd <szilard.parrag@gmail.com>
Build SUCCESS |
Signed-off-by: Parrag Szilárd <szilard.parrag@gmail.com>
Signed-off-by: Parrag Szilárd <szilard.parrag@gmail.com>
The |
Build SUCCESS |
Thanks. |
This PR adds
workers()
support for theredis()
destination driver.This increases the throughput of the Redis destination driver.