CalvinConfluent commented on code in PR #13489:
URL: https://github.com/apache/kafka/pull/13489#discussion_r1159266249
##########
core/src/test/scala/unit/kafka/cluster/PartitionTest.scala:
##########
@@ -1367,6 +1368,10 @@ class PartitionTest extends AbstractPartitionTest {
assertEquals(alterPartitionManager.isrUpdates.size, 1)
val isrItem = alterPartitionManager.isrUpdates.head
assertEquals(isrItem.leaderAndIsr.isr, List(brokerId, remoteBrokerId))
+ isrItem.leaderAndIsr.isrWithBrokerEpoch.foreach(brokerState => {
+ // In ZK mode, the broker epochs in the leaderAndIsr should be -1.
Review Comment:
In the PartitionTest, if the test uses default _partition_, then it is
implemented with ZK metadata cache.
--
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
