You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by jg...@apache.org on 2017/09/15 21:13:08 UTC

kafka git commit: MINOR: Disable KafkaAdminClientTest.testHandleTimeout

Repository: kafka
Updated Branches:
  refs/heads/trunk 8fca43222 -> 6055c7498


MINOR: Disable KafkaAdminClientTest.testHandleTimeout

This test is super flaky in the PR builder. https://issues.apache.org/jira/browse/KAFKA-5792 tracks the fix.

Author: Apurva Mehta <ap...@confluent.io>

Reviewers: Jason Gustafson <ja...@confluent.io>

Closes #3877 from apurvam/MINOR-disable-adminclient-timeout-test


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/6055c749
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/6055c749
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/6055c749

Branch: refs/heads/trunk
Commit: 6055c749899a6d5ae618206548afd1f1e83eb65c
Parents: 8fca432
Author: Apurva Mehta <ap...@confluent.io>
Authored: Fri Sep 15 13:51:52 2017 -0700
Committer: Jason Gustafson <ja...@confluent.io>
Committed: Fri Sep 15 13:51:52 2017 -0700

----------------------------------------------------------------------
 .../java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java  | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/6055c749/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
----------------------------------------------------------------------
diff --git a/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java b/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
index 83416c6..4e3d70c 100644
--- a/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
+++ b/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
@@ -47,6 +47,7 @@ import org.apache.kafka.common.resource.ResourceType;
 import org.apache.kafka.common.utils.MockTime;
 import org.apache.kafka.test.TestCondition;
 import org.apache.kafka.test.TestUtils;
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.Timeout;
@@ -310,6 +311,7 @@ public class KafkaAdminClientTest {
     /**
      * Test handling timeouts.
      */
+    @Ignore // The test is flaky. Should be renabled when this JIRA is fixed: https://issues.apache.org/jira/browse/KAFKA-5792
     @Test
     public void testHandleTimeout() throws Exception {
         HashMap<Integer, Node> nodes = new HashMap<>();