Skip to content

Commit

Permalink
[LI-HOTFIX] Ignoring the flaky test shouldListMovingPartitionsThrough…
Browse files Browse the repository at this point in the history
…Api (#110)

TICKET = KAFKA-9253
LI_DESCRIPTION = There seems to be a race condition.
When a ListPartitionReassignment request is processed, the partition reassignment has already
been completed, and thus the ListPartitionReassignmentResponse does not contain data for that
partition (tp2 in the test), which further causes a NullPointerException.

EXIT_CRITERIA = When the fix for the flaky test is committed in upstream and picked up in this repo
  • Loading branch information
gitlw authored Jan 6, 2021
1 parent d16d4d2 commit 4a53cf7
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import kafka.utils.TestUtils._
import kafka.utils.{Logging, TestUtils}
import kafka.zk.{ReassignPartitionsZNode, ZkVersion, ZooKeeperTestHarness}
import org.junit.Assert.{assertEquals, assertFalse, assertTrue}
import org.junit.{After, Before, Test}
import org.junit.{After, Before, Ignore, Test}
import kafka.admin.ReplicationQuotaUtils._
import org.apache.kafka.clients.admin.{Admin, AdminClientConfig, AlterConfigOp, ConfigEntry, NewPartitionReassignment, NewPartitions, PartitionReassignment, AdminClient => JAdminClient}
import org.apache.kafka.common.{TopicPartition, TopicPartitionReplica}
Expand Down Expand Up @@ -798,6 +798,7 @@ class ReassignPartitionsClusterTest extends ZooKeeperTestHarness with Logging {
assertEquals(Seq(100, 101), zkClient.getReplicasForPartition(tp0))
}

@Ignore
@Test
def shouldListMovingPartitionsThroughApi(): Unit = {
startBrokers(Seq(100, 101))
Expand Down

0 comments on commit 4a53cf7

Please sign in to comment.