You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "jolshan (via GitHub)" <gi...@apache.org> on 2023/06/02 16:25:27 UTC

[GitHub] [kafka] jolshan commented on pull request #13675: KAFKA-14462; [14/N] Add PartitionWriter

jolshan commented on PR #13675:
URL: https://github.com/apache/kafka/pull/13675#issuecomment-1574008136

   Hey David -- whenever we update appendRecords we have to modify a ton of tests that mock the method. (Basically add a ton of any(), since the mocker doesn't get default args). There are 283 failing now 😅 
   
   See here for an example of a PR that updates them: https://github.com/apache/kafka/pull/13391/files


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org