You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2021/05/15 16:06:49 UTC

[GitHub] [kafka] lbradstreet commented on a change in pull request #10704: KAFKA-12791: ConcurrentModificationException in AbstractConfig use by KafkaProducer

lbradstreet commented on a change in pull request #10704:
URL: https://github.com/apache/kafka/pull/10704#discussion_r632977199



##########
File path: clients/src/test/java/org/apache/kafka/common/config/AbstractConfigTest.java
##########
@@ -508,6 +509,55 @@ public void testDocumentationOfExpectNull() {
         assertNull(config.documentationOf("xyz"));
     }
 
+    @Test
+    public void testConcurrentUnusedUse() throws InterruptedException {

Review comment:
       I'm not sure this test adds much value from here but it picks up the issue and takes about 1.5s to run so I am happy to leave it in or remove it.




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org