m1a2st commented on code in PR #19259:
URL: https://github.com/apache/kafka/pull/19259#discussion_r2007871072
##
clients/src/test/java/org/apache/kafka/common/config/AbstractConfigTest.java:
##
@@ -114,6 +114,15 @@ public void testOriginalsWithPrefix() {
assertEquals(ex
chia7712 commented on PR #19259:
URL: https://github.com/apache/kafka/pull/19259#issuecomment-2742476082
This is a kind of big fix, so please add a test
--
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
chia7712 merged PR #19259:
URL: https://github.com/apache/kafka/pull/19259
--
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-unsubscr...@kafka.apache
TaiJuWu commented on code in PR #19259:
URL: https://github.com/apache/kafka/pull/19259#discussion_r2008840279
##
clients/src/test/java/org/apache/kafka/common/config/AbstractConfigTest.java:
##
@@ -114,6 +114,15 @@ public void testOriginalsWithPrefix() {
assertEquals(e
TaiJuWu commented on code in PR #19259:
URL: https://github.com/apache/kafka/pull/19259#discussion_r2008840279
##
clients/src/test/java/org/apache/kafka/common/config/AbstractConfigTest.java:
##
@@ -114,6 +114,15 @@ public void testOriginalsWithPrefix() {
assertEquals(e
TaiJuWu commented on PR #19259:
URL: https://github.com/apache/kafka/pull/19259#issuecomment-2743495248
> This is a kind of big fix, so please add a test
Done. Please take a look.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
TaiJuWu opened a new pull request, #19259:
URL: https://github.com/apache/kafka/pull/19259
In past, we have `AbstractConfig#preProcessParsedConfig` but did not use its
return value
This PR fix such issue.
--
This is an automated message from the Apache Git Service.
To respond to th
