Skip to content

MINOR: Fix missing parenthesis in under-min-ISR trace log#22542

Closed
coderbruis wants to merge 0 commit into
apache:trunkfrom
coderbruis:trunk
Closed

MINOR: Fix missing parenthesis in under-min-ISR trace log#22542
coderbruis wants to merge 0 commit into
apache:trunkfrom
coderbruis:trunk

Conversation

@coderbruis

@coderbruis coderbruis commented Jun 11, 2026

Copy link
Copy Markdown

This PR fixes a missing closing parenthesis in an under-min-ISR trace
log message.

Before: Not increasing HWM because partition is under min
ISR(ISR=Set(1, 2)

After: Not increasing HWM because partition is under min ISR(ISR=Set(1,
2))

This is a log message cleanup only. No behavior change.

Reviewers: Ming-Yen Chung mingyen066@gmail.com, Chia-Ping Tsai chia7712@gmail.com, majialong majialoong@gmail.com, Ken Huang s7133700@gmail.com, Eric Chang e850506@gmail.com, Russole Chen 850905junior@gmail.com

@github-actions github-actions Bot added triage PRs from the community core Kafka Broker small Small PRs labels Jun 11, 2026
@mingyen066

mingyen066 commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

Thank you for the patch. LGTM.

@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

@majialoong majialoong left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the patch. LGTM.

@unknowntpo

Copy link
Copy Markdown
Contributor

LGTM

@github-actions github-actions Bot removed the triage PRs from the community label Jun 13, 2026

@Russole Russole left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved core Kafka Broker small Small PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants