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

Support multiline logs in logstash/log fileset of Filebeat #8562

Merged

Conversation

kvch
Copy link
Contributor

@kvch kvch commented Oct 4, 2018

Multiline JDBC plugin logs were not parsed correctly. From now on the module is capable of aggregating log lines into a single multiline event and its pipeline can parse it correctly.

@kvch kvch requested a review from ruflin October 4, 2018 08:22
@kvch kvch added the needs_backport PR is waiting to be backported to other branches. label Oct 4, 2018
Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Can you add a CHANGELOG entry?

@kvch kvch changed the title Support multiline logs in Filebeat logstash/log fileset Support multiline logs in logstash/log fileset of Filebeat Oct 4, 2018
@kvch kvch merged commit 7271c55 into elastic:master Oct 5, 2018
kvch added a commit to kvch/beats that referenced this pull request Oct 5, 2018
)

Multiline JDBC plugin logs were not parsed correctly. From now on the module is capable of aggregating log lines into a single multiline event and its pipeline can parse it correctly.
(cherry picked from commit 7271c55)
@kvch kvch added v6.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Oct 5, 2018
kvch added a commit that referenced this pull request Oct 5, 2018
…8582)

Multiline JDBC plugin logs were not parsed correctly. From now on the module is capable of aggregating log lines into a single multiline event and its pipeline can parse it correctly.
(cherry picked from commit 7271c55)
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