rajinisivaram commented on a change in pull request #9382:
URL: https://github.com/apache/kafka/pull/9382#discussion_r511051808



##########
File path: core/src/main/scala/kafka/server/ReplicaManager.scala
##########
@@ -770,7 +770,7 @@ class ReplicaManager(val config: KafkaConfig,
             logManager.abortAndPauseCleaning(topicPartition)
 
             val initialFetchState = 
InitialFetchState(BrokerEndPoint(config.brokerId, "localhost", -1),
-              partition.getLeaderEpoch, futureLog.highWatermark)
+              partition.getLeaderEpoch, futureLog.highWatermark, 
lastFetchedEpoch = None)

Review comment:
       I had reset this because 
`ReassignPartitionsIntegrationTest.testAlterLogDirReassignmentThrottle` was 
failing consistently when the offset was test. Having spent a whole day looking 
at what I had broken, I think it is an existing issue. The test itself looks 
new and I can recreate it on trunk. Looks like 
https://issues.apache.org/jira/browse/KAFKA-9087. Will look into that 
separately. For now, I have updated this path.




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to