dajac commented on code in PR #17444: URL: https://github.com/apache/kafka/pull/17444#discussion_r1820921291
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/Utils.java: ########## @@ -235,4 +238,20 @@ public static ApiMessage messageOrNull(ApiMessageAndVersion apiMessageAndVersion return apiMessageAndVersion.message(); } } + + public static long hashSubscriptionMetadata(Map<String, TopicMetadata> subscriptionMetadata) { Review Comment: Thanks. Let me think about it for a while. I do have one first comment though. When we replay records, we should only replay the persisted state. We should not compute hashes there based on the metadata image there. -- 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