Skip to content

Conversation

@mingyen066
Copy link
Collaborator

@mingyen066 mingyen066 commented Nov 6, 2025

As
discussed,
the --fetch-size parameter description in ConsumerPerformance is
misleading. It actually sets max.partition.fetch.bytes (per-partition
limit), not the total fetch size per request.

Reviewers: Ken Huang [email protected], PoAn Yang
[email protected], Yung [email protected], Chia-Ping Tsai
[email protected]

@github-actions github-actions bot added triage PRs from the community tools small Small PRs labels Nov 6, 2025
Copy link
Member

@FrankYang0529 FrankYang0529 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the patch.

@github-actions github-actions bot removed the triage PRs from the community label Nov 7, 2025
Copy link
Member

@chia7712 chia7712 left a comment

Choose a reason for hiding this comment

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

LGTM

@chia7712
Copy link
Member

chia7712 commented Nov 9, 2025

@mingyen066 could you please also update the following file ?

"fetch-size", "The amount of data to fetch in a single request."

@mingyen066
Copy link
Collaborator Author

@chia7712 Sure, updated.

@chia7712 chia7712 merged commit 721cc0c into apache:trunk Nov 11, 2025
20 checks passed
eduwercamacaro pushed a commit to littlehorse-enterprises/kafka that referenced this pull request Nov 12, 2025
)

As [discussed](https://github.com/apache/kafka/pull/18415/files#r2495342682),
the `--fetch-size` parameter description in `ConsumerPerformance` is
misleading. It actually sets `max.partition.fetch.bytes` (per-partition
limit), not the total fetch size per request.

Reviewers: Ken Huang <[email protected]>, PoAn Yang
 <[email protected]>, Yung <[email protected]>, Chia-Ping Tsai
 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants