cmccabe commented on code in PR #18566:
URL: https://github.com/apache/kafka/pull/18566#discussion_r1929425036
##########
tools/src/test/java/org/apache/kafka/tools/reassign/ReassignPartitionsCommandTest.java:
##########
@@ -144,29 +143,6 @@ public void testReassignmentWithAlterPartitionDisabled()
throws Exception {
executeAndVerifyReassignment();
}
- @ClusterTests({
- @ClusterTest(types = {Type.KRAFT, Type.CO_KRAFT}, serverProperties = {
- @ClusterConfigProperty(id = 1, key =
INTER_BROKER_PROTOCOL_VERSION_CONFIG, value = "3.3-IV0"),
- @ClusterConfigProperty(id = 2, key =
INTER_BROKER_PROTOCOL_VERSION_CONFIG, value = "3.3-IV0"),
- @ClusterConfigProperty(id = 3, key =
INTER_BROKER_PROTOCOL_VERSION_CONFIG, value = "3.3-IV0"),
- })
- })
- public void testReassignmentCompletionDuringPartialUpgrade() throws
Exception {
Review Comment:
This was a test for some zk mode functionality.
```
- // Test reassignment during a partial upgrade when some brokers are
relying on
- // `AlterPartition` and some rely on the old notification logic
through Zookeeper.
```
--
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.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
