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

docs(lua transform): Add a guide about merging multi-line events #2354

Merged
2 commits merged into from
Apr 16, 2020

Conversation

ghost
Copy link

@ghost ghost commented Apr 16, 2020

Signed-off-by: Alexander Rodin <rodin.alexander@gmail.com>
@ghost ghost requested a review from binarylogic April 16, 2020 18:41
Signed-off-by: Alexander Rodin <rodin.alexander@gmail.com>
Copy link
Contributor

@binarylogic binarylogic left a comment

Choose a reason for hiding this comment

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

I don't have any constructive feedback, this is a really nice guide and I'm happy we have it.

It avoids issues like fluent/fluent-bit#337.

In the end, when the state holds enough data to extract all fields, a merged event is emitted and
the state is emptied. Then the process repeats as new events arrive.

## Safety Checks
Copy link
Contributor

Choose a reason for hiding this comment

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

I really like this section. Nice work! This is where similar Lua scripts fall short.

@ghost ghost merged commit 19bff5c into master Apr 16, 2020
@binarylogic binarylogic deleted the add-a-guide-about-merging-multiline-events branch April 24, 2020 20:37
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merging logs with Lua guide
2 participants