Skip to content

MINOR: Fix typo in DynamicBrokerConfigTest#22352

Merged
FrankYang0529 merged 1 commit into
apache:trunkfrom
FrankYang0529:minor-fix-typo-in-DynamicBrokerConfigTest
May 23, 2026
Merged

MINOR: Fix typo in DynamicBrokerConfigTest#22352
FrankYang0529 merged 1 commit into
apache:trunkfrom
FrankYang0529:minor-fix-typo-in-DynamicBrokerConfigTest

Conversation

@FrankYang0529
Copy link
Copy Markdown
Member

@FrankYang0529 FrankYang0529 commented May 23, 2026

props -> props3

Reviewers: Chia-Ping Tsai chia7712@gmail.com

@github-actions github-actions Bot added core Kafka Broker tests Test fixes (including flaky tests) small Small PRs labels May 23, 2026
val props3 = new Properties()
props3.put(RemoteLogManagerConfig.REMOTE_LOG_READER_THREADS_PROP, "-1")
val props = new Properties()
props.put(RemoteLogManagerConfig.REMOTE_LOG_READER_THREADS_PROP, "-1")
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.

For consistency, we should use the name prop3, right?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry for that. I misunderstand original comment.

Signed-off-by: PoAn Yang <payang@apache.org>
@FrankYang0529 FrankYang0529 force-pushed the minor-fix-typo-in-DynamicBrokerConfigTest branch from 1505aff to 15f3786 Compare May 23, 2026 05:30
Copy link
Copy Markdown
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

@FrankYang0529 FrankYang0529 merged commit 1bcb231 into apache:trunk May 23, 2026
18 checks passed
@FrankYang0529 FrankYang0529 deleted the minor-fix-typo-in-DynamicBrokerConfigTest branch May 23, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Kafka Broker small Small PRs tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants