Re: [PR] KAFKA-18604; Update transaction coordinator [kafka]

2025-01-20 Thread via GitHub
dajac merged PR #18636: URL: https://github.com/apache/kafka/pull/18636 -- 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.or

Re: [PR] KAFKA-18604; Update transaction coordinator [kafka]

2025-01-20 Thread via GitHub
dajac commented on code in PR #18636: URL: https://github.com/apache/kafka/pull/18636#discussion_r1922713369 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionCoordinatorRecordSerde.java: ## @@ -16,32 +16,29 @@ */ package org.apache.

Re: [PR] KAFKA-18604; Update transaction coordinator [kafka]

2025-01-20 Thread via GitHub
AndrewJSchofield commented on code in PR #18636: URL: https://github.com/apache/kafka/pull/18636#discussion_r1922494231 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionCoordinatorRecordSerde.java: ## @@ -16,32 +16,29 @@ */ package

[PR] KAFKA-18604; Update transaction coordinator [kafka]

2025-01-20 Thread via GitHub
dajac opened a new pull request, #18636: URL: https://github.com/apache/kafka/pull/18636 This patch updates the transaction coordinator record to use the new coordinator record definition. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation