[jira] [Commented] (KAFKA-18930) KRaft MigrationEvent won't retry when failing to write data to ZK

2025-03-08 Thread Luke Chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17932938#comment-17932938 ] Luke Chen commented on KAFKA-18930: --- [~davidarthur] [~mumrah] , I'd like to hear your

Re: [PR] KAFKA-18943: Kafka Streams incorrectly commits TX during task revokation [kafka]

2025-03-08 Thread via GitHub
mjsax commented on code in PR #19164: URL: https://github.com/apache/kafka/pull/19164#discussion_r1986158617 ## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ## @@ -1163,10 +1166,12 @@ void handleRevocation(final Collection revokedPartitio

Re: [PR] MINOR Improve PR linter output [kafka]

2025-03-08 Thread via GitHub
mumrah commented on PR #19159: URL: https://github.com/apache/kafka/pull/19159#issuecomment-2707179546 Example step summary: Commit will look like: ``` MINOR: Disallow unused local variables Recently, we found a regression that could have been detected by static analy

[jira] [Commented] (KAFKA-18942) Add reviewers to PR body with committer-tools

2025-03-08 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17933576#comment-17933576 ] Chia-Ping Tsai commented on KAFKA-18942: +1 to this idea. [~mingyen066] will tak

Re: [PR] MINOR KIP link change to use immutable link [kafka]

2025-03-08 Thread via GitHub
m1a2st commented on PR #19153: URL: https://github.com/apache/kafka/pull/19153#issuecomment-2708602615 ``` https://cwiki.apache.org/confluence/display/KAFKA/KIP-714%3A+Client+metrics+and+observability#KIP714:Clientmetricsandobservability-= ``` Keep only this URL, as it will redir

Re: [PR] MINOR KIP link change to use immutable link [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on PR #19153: URL: https://github.com/apache/kafka/pull/19153#issuecomment-2708606376 > Keep only this URL, as it will redirect to the specific section. how about https://cwiki.apache.org/confluence/x/2xRRCg#KIP714:Clientmetricsandobservability-Clientidentificationa

Re: [PR] KAFKA-18700: Migrate SnapshotPath and Entry in LogHistory to record classes [kafka]

2025-03-08 Thread via GitHub
chia7712 merged PR #19062: URL: https://github.com/apache/kafka/pull/19062 -- 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

[jira] [Updated] (KAFKA-18700) Migrate SnapshotPath, Entry, OffsetAndEpoch, LogFetchInfo, and LogAppendInfo to record classes

2025-03-08 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai updated KAFKA-18700: --- Summary: Migrate SnapshotPath, Entry, OffsetAndEpoch, LogFetchInfo, and LogAppendInfo to rec

Re: [PR] MINOR: Rewrite unchecked operations in Mock API [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on code in PR #19071: URL: https://github.com/apache/kafka/pull/19071#discussion_r1986194037 ## streams/src/test/java/org/apache/kafka/streams/kstream/internals/KTableImplTest.java: ## @@ -391,24 +386,20 @@ public void testStateStore() { public void shou

[jira] [Assigned] (KAFKA-18942) Add reviewers to PR body with committer-tools

2025-03-08 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18942?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai reassigned KAFKA-18942: -- Assignee: Ming-Yen Chung > Add reviewers to PR body with committer-tools > --

[jira] [Resolved] (KAFKA-18700) Migrate SnapshotPath, Entry, OffsetAndEpoch, LogFetchInfo, and LogAppendInfo to record classes

2025-03-08 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai resolved KAFKA-18700. Fix Version/s: 4.1.0 Resolution: Fixed > Migrate SnapshotPath, Entry, OffsetAndEpoc

[jira] [Created] (KAFKA-18944) Remove unused setters from ClusterConfig

2025-03-08 Thread Chia-Ping Tsai (Jira)
Chia-Ping Tsai created KAFKA-18944: -- Summary: Remove unused setters from ClusterConfig Key: KAFKA-18944 URL: https://issues.apache.org/jira/browse/KAFKA-18944 Project: Kafka Issue Type: Impr

Re: [PR] KAFKA-10731: add support for SSL hot reload [kafka]

2025-03-08 Thread via GitHub
github-actions[bot] commented on PR #17987: URL: https://github.com/apache/kafka/pull/17987#issuecomment-2708651331 This PR is being marked as stale since it has not had any activity in 90 days. If you would like to keep this PR alive, please leave a comment asking for a review. If the P

[jira] [Commented] (KAFKA-18420) Find out the license which is in the license file but is not in distribution

2025-03-08 Thread kangning.li (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17933584#comment-17933584 ] kangning.li commented on KAFKA-18420: - file: [https://github.com/apache/kafka/pull/1

Re: [PR] MINOR KIP link change to use immutable link [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on PR #19153: URL: https://github.com/apache/kafka/pull/19153#issuecomment-2708599138 ![螢幕快照 2025-03-09 09-08-00](https://github.com/user-attachments/assets/97bc7361-7379-45cd-af6e-69da15bb1198) @m1a2st could you please fix above links as well? -- This is an auto

Re: [PR] KAFKA-18915: Migrate AdminClientRebootstrapTest to use new test infra [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on code in PR #19094: URL: https://github.com/apache/kafka/pull/19094#discussion_r1986207973 ## core/src/test/java/kafka/test/api/AdminClientRebootstrapTest.java: ## @@ -0,0 +1,111 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more +

[jira] [Commented] (KAFKA-18944) Remove unused setters from ClusterConfig

2025-03-08 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18944?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17933582#comment-17933582 ] Chia-Ping Tsai commented on KAFKA-18944: a example of misusing: https://github.

Re: [PR] KAFKA-18933 Add client integration tests module [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on code in PR #19144: URL: https://github.com/apache/kafka/pull/19144#discussion_r1986210326 ## clients/integration-tests/src/test/java/org/apache/kafka/clients/admin/AdminClientRebootstrapTest.java: ## @@ -33,6 +32,9 @@ import java.util.concurrent.TimeUnit;

Re: [PR] KAFKA-18909: Move DynamicThreadPool to server module [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on PR #19081: URL: https://github.com/apache/kafka/pull/19081#issuecomment-2708604833 @clarkwtc could you please merge trunk to run CI again? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] MINOR: Cleanup connect runtime module [kafka]

2025-03-08 Thread via GitHub
wernerdv commented on code in PR #18074: URL: https://github.com/apache/kafka/pull/18074#discussion_r1986122644 ## connect/runtime/src/main/java/org/apache/kafka/connect/util/KafkaBasedLog.java: ## @@ -466,16 +466,16 @@ protected boolean readPartition(TopicPartition topicPartit

Re: [PR] KAFKA-14484: Move UnifiedLog to storage module [kafka]

2025-03-08 Thread via GitHub
mimaison commented on PR #19030: URL: https://github.com/apache/kafka/pull/19030#issuecomment-2708369200 Rebased on trunk and ported https://github.com/apache/kafka/commit/40db001588047201a406ebe969d1d7d2d5eefd57 -- This is an automated message from the Apache Git Service. To respond to t

Re: [PR] MINOR: Cleanup connect runtime module [kafka]

2025-03-08 Thread via GitHub
ashrivastava88 commented on code in PR #18074: URL: https://github.com/apache/kafka/pull/18074#discussion_r1986112509 ## connect/runtime/src/main/java/org/apache/kafka/connect/util/KafkaBasedLog.java: ## @@ -466,16 +466,16 @@ protected boolean readPartition(TopicPartition topic

Re: [PR] MINOR: Adjust ToC of zk2kraft and fix wrong section number of docker [kafka]

2025-03-08 Thread via GitHub
chia7712 merged PR #19146: URL: https://github.com/apache/kafka/pull/19146 -- 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

Re: [PR] KAFKA-18648: Make `records` in `FetchResponse` nullable again [kafka]

2025-03-08 Thread via GitHub
ijuma commented on PR #19131: URL: https://github.com/apache/kafka/pull/19131#issuecomment-2704563232 Cherry-picked to 4.0. -- 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.

Re: [PR] KAFKA-18943: Kafka Streams incorrectly commits TX during task revokation [kafka]

2025-03-08 Thread via GitHub
mjsax commented on code in PR #19164: URL: https://github.com/apache/kafka/pull/19164#discussion_r1986165537 ## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ## @@ -1153,8 +1157,7 @@ void handleRevocation(final Collection revokedPartitions

[jira] [Updated] (KAFKA-2939) Make AbstractConfig.logUnused() tunable for clients

2025-03-08 Thread Matthias J. Sax (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-2939?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias J. Sax updated KAFKA-2939: --- Labels: (was: newbie) > Make AbstractConfig.logUnused() tunable for clients > -

Re: [PR] KAFKA-18943: Kafka Streams incorrectly commits TX during task revokation [kafka]

2025-03-08 Thread via GitHub
mjsax commented on code in PR #19164: URL: https://github.com/apache/kafka/pull/19164#discussion_r1986165537 ## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ## @@ -1153,8 +1157,7 @@ void handleRevocation(final Collection revokedPartitions

[PR] KAFKA-18944: Remove unused setters from ClusterConfig [kafka]

2025-03-08 Thread via GitHub
clarkwtc opened a new pull request, #19166: URL: https://github.com/apache/kafka/pull/19166 Remove unused `saslServerProperties`, `saslClientProperties`, `adminClientProperties`, `producerProperties`, and `consumerProperties` in ClusterConfig. First, I quickly fixed the unused adminC

[jira] [Assigned] (KAFKA-18944) Remove unused setters from ClusterConfig

2025-03-08 Thread Wei-Ting Chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18944?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Wei-Ting Chen reassigned KAFKA-18944: - Assignee: Wei-Ting Chen (was: Chia-Ping Tsai) > Remove unused setters from ClusterConf

Re: [PR] KAFKA-18909: Move DynamicThreadPool to server module [kafka]

2025-03-08 Thread via GitHub
clarkwtc commented on PR #19081: URL: https://github.com/apache/kafka/pull/19081#issuecomment-2708609581 @chia7712 No problem. I've merged it. -- 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

[jira] [Updated] (KAFKA-18422) add Kafka client upgrade path

2025-03-08 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-18422: Fix Version/s: 4.0.0 > add Kafka client upgrade path > - > >

Re: [PR] KAFKA-16758: Extend Consumer#close with an option to leave the group or not [kafka]

2025-03-08 Thread via GitHub
frankvicky commented on code in PR #17614: URL: https://github.com/apache/kafka/pull/17614#discussion_r1986090241 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/AsyncKafkaConsumer.java: ## @@ -427,7 +428,7 @@ public void onGroupAssignmentUpdated(Set partit

Re: [PR] KAFKA-14484: Move UnifiedLog to storage module [kafka]

2025-03-08 Thread via GitHub
mimaison commented on code in PR #19030: URL: https://github.com/apache/kafka/pull/19030#discussion_r1986107143 ## storage/src/main/java/org/apache/kafka/storage/internals/log/UnifiedLog.java: ## @@ -55,6 +113,2289 @@ public class UnifiedLog { public static final String STR

Re: [PR] KAFKA-18602: Incorrect FinalizedVersionLevel reported for dynamic KRaft quorum [kafka]

2025-03-08 Thread via GitHub
chia7712 commented on code in PR #18685: URL: https://github.com/apache/kafka/pull/18685#discussion_r1986118137 ## core/src/main/scala/kafka/server/metadata/KRaftMetadataCache.scala: ## @@ -522,10 +522,14 @@ class KRaftMetadataCache( if (kraftVersionLevel > 0) { fina

Re: [PR] MINOR: Clean up metadata module [kafka]

2025-03-08 Thread via GitHub
mumrah commented on PR #19069: URL: https://github.com/apache/kafka/pull/19069#issuecomment-2708307265 @sjhajharia can you merge trunk into this branch? There are some recent CI changes that are needed. I will try to review this over the weekend. -- This is an automated message from the A

[jira] [Created] (KAFKA-18931) Debug the case where kafka-share-groups.sh --describe --members does not match with the reality when share consumer is closed during broker outage

2025-03-08 Thread Chirag Wadhwa (Jira)
Chirag Wadhwa created KAFKA-18931: - Summary: Debug the case where kafka-share-groups.sh --describe --members does not match with the reality when share consumer is closed during broker outage Key: KAFKA-18931 URL

[PR] KAFKA-18933 Add client integration tests module [kafka]

2025-03-08 Thread via GitHub
mumrah opened a new pull request, #19144: URL: https://github.com/apache/kafka/pull/19144 Adds a new ":clients:integration-test" Gradle module with one example test. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] KAFKA-18195: Enter incompatible instead of leaving incompatible entires blank in Kafka Streams broker compatibility matrix [kafka]

2025-03-08 Thread via GitHub
mjsax commented on code in PR #18258: URL: https://github.com/apache/kafka/pull/18258#discussion_r1984230751 ## docs/streams/upgrade-guide.html: ## @@ -1735,73 +1735,38 @@ Streams API broker compatibility -The following table shows which versions of the Kafka Stre

Re: [PR] KAFKA-18736: Decide when a heartbeat should be sent [kafka]

2025-03-08 Thread via GitHub
lucasbru commented on code in PR #19121: URL: https://github.com/apache/kafka/pull/19121#discussion_r1983675477 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/StreamsGroupHeartbeatRequestManager.java: ## @@ -254,34 +284,131 @@ public StreamsGroupHeartbeatRe

Re: [PR] KAFKA-18943: Kafka Streams incorrectly commits TX during task revokation [kafka]

2025-03-08 Thread via GitHub
lucasbru commented on code in PR #19164: URL: https://github.com/apache/kafka/pull/19164#discussion_r1986031099 ## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ## @@ -1153,8 +1157,7 @@ void handleRevocation(final Collection revokedPartiti

[jira] [Updated] (KAFKA-18074) Add kafka client compatibility matrix

2025-03-08 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-18074: Fix Version/s: (was: 4.0.0) > Add kafka client compatibility matrix >

Re: [PR] KAFKA-18943: Kafka Streams incorrectly commits TX during task revokation [kafka]

2025-03-08 Thread via GitHub
cadonna commented on code in PR #19164: URL: https://github.com/apache/kafka/pull/19164#discussion_r1986072577 ## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ## @@ -1163,10 +1166,12 @@ void handleRevocation(final Collection revokedPartit

[jira] [Updated] (KAFKA-18074) Add kafka client compatibility matrix

2025-03-08 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-18074: Fix Version/s: 4.0.0 > Add kafka client compatibility matrix > ---

Re: [PR] MINOR: Clean up metadata module [kafka]

2025-03-08 Thread via GitHub
sjhajharia commented on PR #19069: URL: https://github.com/apache/kafka/pull/19069#issuecomment-2708215564 cc: @m1a2st / @mumrah for a final review Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[jira] [Updated] (KAFKA-18422) add Kafka client upgrade path

2025-03-08 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-18422: Fix Version/s: (was: 4.0.0) > add Kafka client upgrade path > - >

Re: [PR] KAFKA-18602: Incorrect FinalizedVersionLevel reported for dynamic KRaft quorum [kafka]

2025-03-08 Thread via GitHub
FrankYang0529 commented on code in PR #18685: URL: https://github.com/apache/kafka/pull/18685#discussion_r1986040272 ## core/src/main/scala/kafka/server/metadata/KRaftMetadataCache.scala: ## @@ -522,10 +522,14 @@ class KRaftMetadataCache( if (kraftVersionLevel > 0) {

Re: [PR] KAFKA-18873: Fixed incorrect error message when exceeds 5 for transactional producers [kafka]

2025-03-08 Thread via GitHub
EsMoX commented on PR #19041: URL: https://github.com/apache/kafka/pull/19041#issuecomment-2708346593 > @EsMoX I just noticed there's a superfluous newline in the error message in the PR, can you remove it please? Thanks! @kirktrue Done, Thank you! -- This is an automated message

Re: [PR] KAFKA-18927: Remove LATEST_0_11, LATEST_1_0, LATEST_1_1, LATEST_2_0 [kafka]

2025-03-08 Thread via GitHub
Parkerhiphop commented on PR #19134: URL: https://github.com/apache/kafka/pull/19134#issuecomment-2708352912 Thanks for the testing and information provided. I will take a look at these failed tests. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] KAFKA-18736: Decide when a heartbeat should be sent [kafka]

2025-03-08 Thread via GitHub
cadonna commented on code in PR #19121: URL: https://github.com/apache/kafka/pull/19121#discussion_r1982894114 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/StreamsGroupHeartbeatRequestManager.java: ## @@ -254,34 +284,131 @@ public StreamsGroupHeartbeatReq