Skip to content

Conversation

cosmo0920
Copy link
Contributor

This is corresponding to fluent/fluent-bit#10653.

@cosmo0920 cosmo0920 requested review from a team as code owners October 7, 2025 09:25
@cosmo0920 cosmo0920 force-pushed the cosmo0920-add-description-for-buffer-limit-of-multiline branch from a22fd3a to e48eb29 Compare October 7, 2025 09:28
Copy link
Contributor

@esmerel esmerel left a comment

Choose a reason for hiding this comment

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

Looks good to me

multiline_buffer_limit 2MiB # default; limit concatenated multiline message size
```

If the limit is reached, Fluent Bit flushes the partial record with `multiline_truncated: true` metadata immediately to prevent Out-Of-Memory (OOM) conditions.
Copy link
Contributor

@lecaros lecaros Oct 7, 2025

Choose a reason for hiding this comment

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

does this limit work as a soft-limit for the current record being appended? or is the ml record flushed without appending the current record?
what happens to the remaining records intended to be appended to the ml record? are they discarded or flushed as individual records?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

does this limit work as a soft-limit for the current record being appended? or is the ml record flushed without appending the current record?
what happens to the remaining records intended to be appended to the ml record? are they discarded or flushed as individual records?

This works as a hard limit and the remaining records will not be concatenated.
They will be flush as individual record(s).

@eschabell
Copy link
Collaborator

Merging the doc changes that have been approved.

@eschabell eschabell merged commit 86975e9 into master Oct 10, 2025
7 checks passed
@eschabell eschabell deleted the cosmo0920-add-description-for-buffer-limit-of-multiline branch October 10, 2025 10:38
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.

4 participants