You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "NickJavaDev88 (via GitHub)" <gi...@apache.org> on 2026/08/24 17:22:39 UTC
[PR] MINOR: Remove unnecessary props.asScala in DynamicBrokerConfig [kafka]
NickJavaDev88 opened a new pull request, #23259:
URL: https://github.com/apache/kafka/pull/23259
Removed the dead `props.asScala` line.
### Testing
```bash
./gradlew core:test --tests kafka.server.DynamicBrokerConfigTest
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: Remove unnecessary props.asScala in DynamicBrokerConfig [kafka]
Posted by "m1a2st (via GitHub)" <gi...@apache.org>.
m1a2st merged PR #23259:
URL: https://github.com/apache/kafka/pull/23259
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] MINOR: Remove unnecessary props.asScala in DynamicBrokerConfig [kafka]
Posted by "NickJavaDev88 (via GitHub)" <gi...@apache.org>.
NickJavaDev88 commented on PR #23259:
URL: https://github.com/apache/kafka/pull/23259#issuecomment-5404160009
Thanks both! 🚀 @chia7712 for the guidance throughout the review, and @m1a2st for reviewing and merging. Appreciate the time you both put into this! 🙌
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org