[GitHub] [kafka] YiDing-Duke commented on a change in pull request #10843: MINOR: Log formatting for exceptions during configuration related operations

2021-06-14 Thread GitBox
YiDing-Duke commented on a change in pull request #10843: URL: https://github.com/apache/kafka/pull/10843#discussion_r651361653 ## File path: clients/src/main/java/org/apache/kafka/clients/admin/ConfigEntry.java ## @@ -174,11 +174,15 @@ public int hashCode() { return

[GitHub] [kafka] YiDing-Duke commented on a change in pull request #10843: MINOR: Log formatting for exceptions during configuration related operations

2021-06-14 Thread GitBox
YiDing-Duke commented on a change in pull request #10843: URL: https://github.com/apache/kafka/pull/10843#discussion_r651094578 ## File path: core/src/main/scala/kafka/server/DynamicBrokerConfig.scala ## @@ -469,7 +469,7 @@ class DynamicBrokerConfig(private val kafkaConfig:

[GitHub] [kafka] YiDing-Duke commented on a change in pull request #10843: MINOR: Log formatting for exceptions during configuration related operations

2021-06-14 Thread GitBox
YiDing-Duke commented on a change in pull request #10843: URL: https://github.com/apache/kafka/pull/10843#discussion_r651094578 ## File path: core/src/main/scala/kafka/server/DynamicBrokerConfig.scala ## @@ -469,7 +469,7 @@ class DynamicBrokerConfig(private val kafkaConfig:

[GitHub] [kafka] YiDing-Duke commented on a change in pull request #10843: MINOR: Log formatting for exceptions during configuration related operations

2021-06-14 Thread GitBox
YiDing-Duke commented on a change in pull request #10843: URL: https://github.com/apache/kafka/pull/10843#discussion_r651094904 ## File path: clients/src/main/java/org/apache/kafka/clients/admin/ConfigEntry.java ## @@ -174,11 +174,15 @@ public int hashCode() { return

[GitHub] [kafka] YiDing-Duke commented on a change in pull request #10843: MINOR: Log formatting for exceptions during configuration related operations

2021-06-14 Thread GitBox
YiDing-Duke commented on a change in pull request #10843: URL: https://github.com/apache/kafka/pull/10843#discussion_r651094578 ## File path: core/src/main/scala/kafka/server/DynamicBrokerConfig.scala ## @@ -469,7 +469,7 @@ class DynamicBrokerConfig(private val kafkaConfig: