kamalcph commented on PR #15825: URL: https://github.com/apache/kafka/pull/15825#issuecomment-2122625313
> @kamalcph , had another look, left some comments. Also, could we add an integration test for this case like you described in the PR description? `FetchFromFollowerIntegrationTest` should be a good place to do that. There are some examples to shutdown nodes and then restart it, and then verify producer/consumer still work well. Added `FetchFromLeaderWithCorruptedCheckpointTest` integration test and ensured that the produce fails without the patch (ran the test against trunk and this branch). -- 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
