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

Backport logstash concurrent event modification fix for #1410 #1438

Merged
merged 1 commit into from
Apr 21, 2016

Conversation

urso
Copy link

@urso urso commented Apr 20, 2016

logstash output used to add metadata to an event. With potentially having
multiple outputs configured this is some bad practice. Instead the logstash
output plugin will add '@metadata' to the event when encoding and only if
'@metadata' is not already present in event

logstash output used to add metadata to an event. With potentially having
multiple outputs configured this is some bad practice. Instead the logstash
output plugin will add '@metadata' to the event when encoding and only if
'@metadata' is not already present in event
@tsg
Copy link
Contributor

tsg commented Apr 21, 2016

Thanks @urso!

@tsg tsg merged commit 448a46d into elastic:1.2 Apr 21, 2016
@urso urso deleted the backport/1428-logstash-event-meta branch May 30, 2016 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants