chia7712 commented on code in PR #14849: URL: https://github.com/apache/kafka/pull/14849#discussion_r2207532864
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/runtime/CoordinatorPlayback.java: ########## @@ -38,4 +38,18 @@ void replay( short producerEpoch, U record ) throws RuntimeException; + + /** + * Invoke operations when a batch has been successfully loaded. + * + * @param offset the offset of the last record in the batch plus one. + */ + void updateLastWrittenOffset(Long offset); Review Comment: Excuse me, why not use the primitive type `long`? I plan to file a PR to change the type, and please feel free to stop me if I misunderstand anything. -- 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