Skip to content

Bound decompressed record size (#23429) - #23449

Merged
chia7712 merged 1 commit into
apache:4.2from
omkreddy:backport-23429-42
Sep 13, 2026
Merged

chia7712 merged 1 commit into
apache:4.2from
omkreddy:backport-23429-42

Conversation

@omkreddy

@omkreddy omkreddy commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

The fix improves validation and handling of record sizes across the
broker's record processing paths.

A new configurable broker/topic-level setting,
max.decompressed.message.bytes, allows operators to control the maximum
permitted decompressed record size. The limit is applied consistently
during produce validation, log compaction and timestamp-based list
offsets lookups.

Existing APIs remain backward compatible with safe default behavior for
callers that do not explicitly configure a limit.

Reviewers: Chia-Ping Tsai chia7712@gmail.com (cherry picked from
commit 4d781f5) (cherry picked from
commit a544dc8effd8f1bf90c5746130c91b2a8127e9db) (cherry picked from
commit c9cea9f564195771e72f1c9a2c68059bb044e599)

The fix improves validation and handling of record sizes across the
broker's record processing paths.

A new configurable broker/topic-level setting,
max.decompressed.message.bytes, allows operators to control the maximum
permitted decompressed record size. The limit is applied consistently
during produce validation, log compaction and timestamp-based list
offsets lookups.

Existing APIs remain backward compatible with safe default behavior for
callers that do not explicitly configure a limit.

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
(cherry picked from commit 4d781f5)
(cherry picked from commit a544dc8effd8f1bf90c5746130c91b2a8127e9db)
(cherry picked from commit c9cea9f564195771e72f1c9a2c68059bb044e599)

@chia7712 chia7712 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@chia7712
chia7712 merged commit 62e667d into apache:4.2 Sep 13, 2026
13 of 14 checks passed
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.

3 participants