Re: [PR] MINOR: Remove DeleteSegmentsByRetentionTimeTest#executeTieredStorageTest flaky annotation [kafka]

2025-04-04 Thread via GitHub
chia7712 merged PR #19301: URL: https://github.com/apache/kafka/pull/19301 -- 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] [Assigned] (KAFKA-19087) Move TransactionState to transaction-coordinator module

2025-04-04 Thread Cheng Yi Chang (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19087?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Cheng Yi Chang reassigned KAFKA-19087: -- Assignee: Cheng Yi Chang (was: Chia-Ping Tsai) > Move TransactionState to transactio

Re: [PR] MINOR: Update Mockito on 3.9 for Java 23 support [kafka]

2025-04-04 Thread via GitHub
chia7712 merged PR #19367: URL: https://github.com/apache/kafka/pull/19367 -- 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-10551: Add topic id support to produce request and response [kafka]

2025-04-04 Thread via GitHub
OmniaGM commented on PR #15968: URL: https://github.com/apache/kafka/pull/15968#issuecomment-2778290434 > That's great, thanks! @ijuma and @jolshan I fixed the conflicts now -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[jira] [Commented] (KAFKA-19087) Move TransactionState to transaction-coordinator module

2025-04-04 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940938#comment-17940938 ] Chia-Ping Tsai commented on KAFKA-19087: Maybe we should consider leveraging `Tr

Re: [PR] KAFKA-19001: Use streams group-level configurations in heartbeat [kafka]

2025-04-04 Thread via GitHub
lucasbru commented on code in PR #19219: URL: https://github.com/apache/kafka/pull/19219#discussion_r2028472624 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java: ## @@ -8148,7 +8174,10 @@ private TaskAssignor streamsGroupAssignor(St

[jira] [Created] (KAFKA-19088) Consider logging the unknownStateFilters when listing transation ids

2025-04-04 Thread Chia-Ping Tsai (Jira)
Chia-Ping Tsai created KAFKA-19088: -- Summary: Consider logging the unknownStateFilters when listing transation ids Key: KAFKA-19088 URL: https://issues.apache.org/jira/browse/KAFKA-19088 Project: Kaf

Re: [PR] KAFKA-18761: [2/N] List share group offsets with state and auth [kafka]

2025-04-04 Thread via GitHub
AndrewJSchofield merged PR #19328: URL: https://github.com/apache/kafka/pull/19328 -- 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...@kafk

[jira] [Commented] (KAFKA-19088) Consider logging the unknownStateFilters when listing transation ids

2025-04-04 Thread Jhen-Yung Hsu (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940959#comment-17940959 ] Jhen-Yung Hsu commented on KAFKA-19088: --- Hi, I'm working on this, thanks. :) > Co

[jira] [Assigned] (KAFKA-19088) Consider logging the unknownStateFilters when listing transation ids

2025-04-04 Thread Jhen-Yung Hsu (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jhen-Yung Hsu reassigned KAFKA-19088: - Assignee: Jhen-Yung Hsu (was: Chia-Ping Tsai) > Consider logging the unknownStateFilte

[jira] [Created] (KAFKA-19031) Ensure brokers do not return null records in ShareFetchResponse

2025-04-04 Thread Andrew Schofield (Jira)
Andrew Schofield created KAFKA-19031: Summary: Ensure brokers do not return null records in ShareFetchResponse Key: KAFKA-19031 URL: https://issues.apache.org/jira/browse/KAFKA-19031 Project: Kafk

Re: [PR] KAFKA-10551: Add topic id support to produce request and response [kafka]

2025-04-04 Thread via GitHub
OmniaGM commented on PR #15968: URL: https://github.com/apache/kafka/pull/15968#issuecomment-2773053998 > @OmniaGM Would you like someone to help get this over the finish line? I am nearly done with fixing the conflict should publish this this week. sorry for late replay I have been a

Re: [PR] [MINOR] Cleanup Core Module [kafka]

2025-04-04 Thread via GitHub
sjhajharia commented on PR #19372: URL: https://github.com/apache/kafka/pull/19372#issuecomment-2779234357 Thanks @m1a2st for the review. I have addressed the comments and pushed a fresh commit. Pls review when possible. -- This is an automated message from the Apache Git Service. To r

Re: [PR] KAFKA-16729: Support isolation level for share consumer [kafka]

2025-04-04 Thread via GitHub
adixitconfluent commented on PR #19261: URL: https://github.com/apache/kafka/pull/19261#issuecomment-2774680482 > Thanks for the updates. In general, I'm happy with the PR, with the exception of the `filterRecordBatchesFromAcquiredRecords` method. It needs much more comprehensive commenting

[jira] [Created] (KAFKA-19085) SharePartitionManagerTest testMultipleConcurrentShareFetches throws silent exception and works incorrectly

2025-04-04 Thread Abhinav Dixit (Jira)
Abhinav Dixit created KAFKA-19085: - Summary: SharePartitionManagerTest testMultipleConcurrentShareFetches throws silent exception and works incorrectly Key: KAFKA-19085 URL: https://issues.apache.org/jira/browse/K

Re: [PR] KAFKA-18935: Ensure brokers do not return null records in FetchResponse [kafka]

2025-04-04 Thread via GitHub
junrao commented on code in PR #19167: URL: https://github.com/apache/kafka/pull/19167#discussion_r2025673505 ## clients/src/main/java/org/apache/kafka/common/requests/FetchResponse.java: ## @@ -87,7 +87,7 @@ public FetchResponseData data() { * We may also return INCONSIST

[jira] [Commented] (KAFKA-19066) JDK 24 support

2025-04-04 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19066?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940079#comment-17940079 ] Chia-Ping Tsai commented on KAFKA-19066: related to https://github.com/apache/ka

[jira] [Created] (KAFKA-19087) Move TransactionState to transaction-coordinator module

2025-04-04 Thread Chia-Ping Tsai (Jira)
Chia-Ping Tsai created KAFKA-19087: -- Summary: Move TransactionState to transaction-coordinator module Key: KAFKA-19087 URL: https://issues.apache.org/jira/browse/KAFKA-19087 Project: Kafka I

Re: [PR] KAFKA-16718-2/n: KafkaAdminClient and GroupCoordinator implementation for DeleteShareGroupOffsets RPC [kafka]

2025-04-04 Thread via GitHub
chirag-wadhwa5 commented on PR #18976: URL: https://github.com/apache/kafka/pull/18976#issuecomment-2778711584 Hi @AndrewJSchofield, I have resolved the conflicts. The PR is ready for another round of review. Thanks ! -- This is an automated message from the Apache Git Service. To respond

Re: [PR] MINOR cleanup log related test [kafka]

2025-04-04 Thread via GitHub
m1a2st commented on code in PR #19375: URL: https://github.com/apache/kafka/pull/19375#discussion_r2028926018 ## core/src/test/scala/unit/kafka/log/LogManagerTest.scala: ## @@ -147,7 +150,10 @@ class LogManagerTest { assertEquals(1, logManager.liveLogDirs.size) val log

Re: [PR] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029040633 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3664,7 +3664,6 @@ public void testFetcherDontCacheAnyData() {

[PR] KAFKA-19037: Integrate consumer-side code with Streams [kafka]

2025-04-04 Thread via GitHub
cadonna opened a new pull request, #19377: URL: https://github.com/apache/kafka/pull/19377 Delete this text and replace it with a detailed description of your change. The PR title and body will become the squashed commit message. If you would like to tag individuals, add some comm

Re: [PR] MINOR: Use readable interface to parse response [kafka]

2025-04-04 Thread via GitHub
trnguyencflt commented on code in PR #19353: URL: https://github.com/apache/kafka/pull/19353#discussion_r2029055199 ## clients/src/main/java/org/apache/kafka/common/requests/AbstractResponse.java: ## @@ -51,8 +53,8 @@ final ByteBuffer serializeWithHeader(ResponseHeader header,

Re: [PR] MINOR: Use readable interface to parse response [kafka]

2025-04-04 Thread via GitHub
trnguyencflt commented on PR #19353: URL: https://github.com/apache/kafka/pull/19353#issuecomment-2779128261 [nit] Please remove this block "Delete this text " from the description of the PR. Otherwise, LGTM -- This is an automated message from the Apache Git Service. To respon

[jira] [Assigned] (KAFKA-19089) Gradle :test task failing unexpectedly

2025-04-04 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19089?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai reassigned KAFKA-19089: -- Assignee: Chia-Ping Tsai > Gradle :test task failing unexpectedly > -

[PR] MINOR cleanup log related test [kafka]

2025-04-04 Thread via GitHub
m1a2st opened a new pull request, #19375: URL: https://github.com/apache/kafka/pull/19375 Some parameter is unused and fix assertion -- 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

Re: [PR] MINOR: Consider repartition source topic for changelog topics [kafka]

2025-04-04 Thread via GitHub
lucasbru merged PR #19373: URL: https://github.com/apache/kafka/pull/19373 -- 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] [WIP]KAFKA-19080 The constraint on segment.ms is not enforced at topic level [kafka]

2025-04-04 Thread via GitHub
m1a2st commented on PR #19371: URL: https://github.com/apache/kafka/pull/19371#issuecomment-2778790334 Hello @junrao, @chia7712 I have a question about this problem. In [[KAFKA-16368](https://github.com/apache/kafka/pull/18140/files)](https://github.com/apache/kafka/pull/18140/files),

Re: [PR] KAFKA-19074: Remove the cached responseData from ShareFetchResponse [kafka]

2025-04-04 Thread via GitHub
TaiJuWu commented on code in PR #19357: URL: https://github.com/apache/kafka/pull/19357#discussion_r2029013870 ## server/src/test/java/org/apache/kafka/server/share/fetch/ShareFetchTest.java: ## @@ -43,11 +52,15 @@ public class ShareFetchTest { private static final String M

Re: [PR] KAFKA-16684: Remove cache in responseData [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop commented on code in PR #16532: URL: https://github.com/apache/kafka/pull/16532#discussion_r2029035269 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3665,6 +3670,18 @@ public void testWhenFetchResponseReturnsALeaderSh

Re: [PR] KAFKA-16684: Remove cache in responseData [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop commented on code in PR #16532: URL: https://github.com/apache/kafka/pull/16532#discussion_r2029035269 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3665,6 +3670,18 @@ public void testWhenFetchResponseReturnsALeaderSh

[PR] MINOR: Consider repartition source topic for changelog topics [kafka]

2025-04-04 Thread via GitHub
cadonna opened a new pull request, #19373: URL: https://github.com/apache/kafka/pull/19373 With the Streams rebalance protocol, when the partition count for changelog topics are computed the topic manager only considers external source topics and throws if there are no source topics

Re: [PR] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
m1a2st commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029039303 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3664,7 +3664,6 @@ public void testFetcherDontCacheAnyData() { resp

[PR] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop opened a new pull request, #19376: URL: https://github.com/apache/kafka/pull/19376 This is from [#16532's comment](https://github.com/apache/kafka/pull/16532/files#r2028985028): The forEach loop in the assertion will never execute because `nonResponseData` is empty.

Re: [PR] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
chia7712 commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029039198 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3664,7 +3664,6 @@ public void testFetcherDontCacheAnyData() { re

[PR] KAFKA-19030: Remove metricNamePrefix from RequestChannel [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop opened a new pull request, #19374: URL: https://github.com/apache/kafka/pull/19374 As described in the JIRA ticket, controlPlaneRequestChannelOpt was removed from KRaft mode, so there's no need to use the metrics prefix anymore. This change removes `metricNamePrefix` from

Re: [PR] KAFKA-16684: Remove cache in responseData [kafka]

2025-04-04 Thread via GitHub
FrankYang0529 commented on code in PR #16532: URL: https://github.com/apache/kafka/pull/16532#discussion_r2028985028 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3665,6 +3670,18 @@ public void testWhenFetchResponseReturnsALeaderS

Re: [PR] KAFKA-19074: Remove the cached responseData from ShareFetchResponse [kafka]

2025-04-04 Thread via GitHub
FrankYang0529 commented on code in PR #19357: URL: https://github.com/apache/kafka/pull/19357#discussion_r2028994700 ## server/src/test/java/org/apache/kafka/server/share/fetch/ShareFetchTest.java: ## @@ -43,11 +52,15 @@ public class ShareFetchTest { private static final St

Re: [PR] MINOR: ShutdownableThread: log on error level on FatalExitError [kafka]

2025-04-04 Thread via GitHub
soarez merged PR #19351: URL: https://github.com/apache/kafka/pull/19351 -- 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.o

Re: [PR] [MINOR] Cleanup Core Module [kafka]

2025-04-04 Thread via GitHub
m1a2st commented on code in PR #19372: URL: https://github.com/apache/kafka/pull/19372#discussion_r2029065027 ## core/src/main/java/kafka/server/share/SharePartitionManager.java: ## @@ -357,7 +356,7 @@ public CompletableFuture { return isrSize == 0 && elrSize ==

Re: [PR] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029101369 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3664,7 +3664,6 @@ public void testFetcherDontCacheAnyData() {

Re: [PR] MINOR: Backport Gradle upgrades to 3.9 [kafka]

2025-04-04 Thread via GitHub
chia7712 merged PR #19365: URL: https://github.com/apache/kafka/pull/19365 -- 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] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029102952 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3664,7 +3664,6 @@ public void testFetcherDontCacheAnyData() {

Re: [PR] KAFKA-16684: Remove cache in responseData [kafka]

2025-04-04 Thread via GitHub
chia7712 commented on code in PR #16532: URL: https://github.com/apache/kafka/pull/16532#discussion_r2028993496 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3665,6 +3670,18 @@ public void testWhenFetchResponseReturnsALeaderShipCh

[jira] [Created] (KAFKA-19089) Gradle :test task failing unexpectedly

2025-04-04 Thread David Arthur (Jira)
David Arthur created KAFKA-19089: Summary: Gradle :test task failing unexpectedly Key: KAFKA-19089 URL: https://issues.apache.org/jira/browse/KAFKA-19089 Project: Kafka Issue Type: Bug

Re: [PR] KAFKA-7952: use in memory stores for KTable test [kafka]

2025-04-04 Thread via GitHub
mjsax merged PR #19218: URL: https://github.com/apache/kafka/pull/19218 -- 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-16758: Extend Consumer#close with an option to leave the group or not [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on code in PR #17614: URL: https://github.com/apache/kafka/pull/17614#discussion_r2029109982 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractHeartbeatRequestManager.java: ## @@ -182,7 +181,7 @@ public NetworkClientDelegate.PollRe

Re: [PR] KAFKA-19070:: Adding task number to user provided client id to ensure each consumer has a unique client ID to avoid metric registration conflicts. [kafka]

2025-04-04 Thread via GitHub
kumarpritam863 commented on PR #19341: URL: https://github.com/apache/kafka/pull/19341#issuecomment-2779267375 Hi @mimaison can you please review this. 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 ab

Re: [PR] MINOR: rename streams group configs for consistency with other configuration settings. [kafka]

2025-04-04 Thread via GitHub
lucasbru merged PR #19360: URL: https://github.com/apache/kafka/pull/19360 -- 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] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029047826 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3664,7 +3664,6 @@ public void testFetcherDontCacheAnyData() {

Re: [PR] KAFKA-19030: Remove metricNamePrefix from RequestChannel [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop commented on code in PR #19374: URL: https://github.com/apache/kafka/pull/19374#discussion_r2028918289 ## core/src/main/scala/kafka/network/RequestChannel.scala: ## @@ -349,8 +348,8 @@ class RequestChannel(val queueSize: Int, private val requestQueue = new Arra

Re: [PR] KAFKA-19077: Propagate shutdownRequested field [kafka]

2025-04-04 Thread via GitHub
lucasbru commented on code in PR #19359: URL: https://github.com/apache/kafka/pull/19359#discussion_r2028433315 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/streams/StreamsGroup.java: ## @@ -197,6 +197,13 @@ public static class DeadlineAndEpoch { *

Re: [PR] MINOR: Use readable interface to parse response [kafka]

2025-04-04 Thread via GitHub
soondenana commented on code in PR #19353: URL: https://github.com/apache/kafka/pull/19353#discussion_r2029166476 ## clients/src/main/java/org/apache/kafka/common/requests/AbstractResponse.java: ## @@ -51,8 +53,8 @@ final ByteBuffer serializeWithHeader(ResponseHeader header, sh

[PR] KAFKA-19057: Stabilize KIP-932 RPCs for AK 4.1 [kafka]

2025-04-04 Thread via GitHub
AndrewJSchofield opened a new pull request, #19378: URL: https://github.com/apache/kafka/pull/19378 This PR removes the unstable API flag for the KIP-932 RPCs. The 4 RPCs which were exposed for the early access release in AK 4.0 are stabilised at v1. This is because the RPCs have evol

[jira] [Updated] (KAFKA-19073) Add Transactional ID pattern filter to ListTransactions API

2025-04-04 Thread Calvin Liu (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19073?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Calvin Liu updated KAFKA-19073: --- Summary: Add Transactional ID pattern filter to ListTransactions API (was: Add Transactional ID pre

[jira] [Resolved] (KAFKA-18761) kafka-share-groups.sh --describe only shows offset information when members exist

2025-04-04 Thread Andrew Schofield (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18761?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Schofield resolved KAFKA-18761. -- Resolution: Fixed > kafka-share-groups.sh --describe only shows offset information whe

[jira] [Assigned] (KAFKA-18902) Implement ShareConsumer option to throw on poll if there are unacked records

2025-04-04 Thread Andrew Schofield (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18902?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Schofield reassigned KAFKA-18902: Assignee: Shivsundar R (was: Andrew Schofield) > Implement ShareConsumer option

[jira] [Updated] (KAFKA-19089) Gradle :test task failing unexpectedly

2025-04-04 Thread David Arthur (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19089?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Arthur updated KAFKA-19089: - Component/s: build > Gradle :test task failing unexpectedly > --

Re: [PR] KAFKA-18892: KIP-877 Add support for ClientQuotaCallback [kafka]

2025-04-04 Thread via GitHub
m1a2st commented on PR #19068: URL: https://github.com/apache/kafka/pull/19068#issuecomment-2777484507 Thanks for @mimaison review, addressed all comments -- 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 t

Re: [PR] KAFKA-18891: KIP-877 add support for RemoteLogMetadataManager and RemoteStorageManager [kafka]

2025-04-04 Thread via GitHub
TaiJuWu commented on code in PR #19286: URL: https://github.com/apache/kafka/pull/19286#discussion_r2022590789 ## core/src/main/java/kafka/log/remote/RemoteLogManager.java: ## @@ -407,14 +413,17 @@ private void configureRLMM() { rlmmProps.put(LOG_DIR_CONFIG, logDir);

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

2025-04-04 Thread via GitHub
ableegoldman commented on code in PR #17614: URL: https://github.com/apache/kafka/pull/17614#discussion_r2025873637 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/ConsumerHeartbeatRequestManager.java: ## @@ -211,6 +214,28 @@ public ConsumerMembershipManager

Re: [PR] KAFKA-16729: Support isolation level for share consumer [kafka]

2025-04-04 Thread via GitHub
adixitconfluent commented on code in PR #19261: URL: https://github.com/apache/kafka/pull/19261#discussion_r2022946269 ## core/src/main/java/kafka/server/share/SharePartition.java: ## @@ -2484,6 +2504,205 @@ private long startOffsetDuringInitialization(long partitionDataStartOf

Re: [PR] KAFKA-18935: Ensure brokers do not return null records in FetchResponse [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on code in PR #19167: URL: https://github.com/apache/kafka/pull/19167#discussion_r2025822378 ## clients/src/main/java/org/apache/kafka/common/requests/FetchResponse.java: ## @@ -87,7 +87,7 @@ public FetchResponseData data() { * We may also return INCON

[jira] [Commented] (KAFKA-19033) FindCoordinator API always return COORDINATOR_NOT_AVAILABLE for CoordinatorType.SHARE

2025-04-04 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19033?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17937947#comment-17937947 ] David Jacot commented on KAFKA-19033: - I alraedy have a patch for it: https://github

[jira] [Assigned] (KAFKA-18900) Create a consumer config to select between implicit and explicit acknowledgement

2025-04-04 Thread Andrew Schofield (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18900?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Schofield reassigned KAFKA-18900: Assignee: Shivsundar R (was: Andrew Schofield) > Create a consumer config to sel

[PR] Cleanup core 2 [kafka]

2025-04-04 Thread via GitHub
sjhajharia opened a new pull request, #19379: URL: https://github.com/apache/kafka/pull/19379 Delete this text and replace it with a detailed description of your change. The PR title and body will become the squashed commit message. If you would like to tag individuals, add some c

Re: [PR] Cleanup core 2 [kafka]

2025-04-04 Thread via GitHub
sjhajharia closed pull request #19379: Cleanup core 2 URL: https://github.com/apache/kafka/pull/19379 -- 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: ji

Re: [PR] MINOR: Clean up unreachable code in FetcherTest [kafka]

2025-04-04 Thread via GitHub
Parkerhiphop commented on code in PR #19376: URL: https://github.com/apache/kafka/pull/19376#discussion_r2029097843 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3663,8 +3663,7 @@ public void testFetcherDontCacheAnyData() {

Re: [PR] KAFKA-16684: Remove cache in responseData [kafka]

2025-04-04 Thread via GitHub
TaiJuWu commented on code in PR #16532: URL: https://github.com/apache/kafka/pull/16532#discussion_r2029018014 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/FetcherTest.java: ## @@ -3665,6 +3670,18 @@ public void testWhenFetchResponseReturnsALeaderShipCha

[jira] [Commented] (KAFKA-19079) KIP-890 Cleanups

2025-04-04 Thread Justine Olshan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17941041#comment-17941041 ] Justine Olshan commented on KAFKA-19079: [~iamoshione]  KIP-890 is a kafka impr

[jira] [Assigned] (KAFKA-15758) Always schedule wrapped callbacks

2025-04-04 Thread Justine Olshan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15758?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justine Olshan reassigned KAFKA-15758: -- Assignee: David Jacot > Always schedule wrapped callbacks >

[jira] [Updated] (KAFKA-19073) Add Transactional ID prefix filter to ListTransactions API

2025-04-04 Thread Calvin Liu (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19073?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Calvin Liu updated KAFKA-19073: --- Description: https://cwiki.apache.org/confluence/display/KAFKA/KIP-1152%3A+Add+transactional+ID+patt

Re: [PR] KAFKA-18892: KIP-877 Add support for ClientQuotaCallback [kafka]

2025-04-04 Thread via GitHub
Yunyung commented on code in PR #19068: URL: https://github.com/apache/kafka/pull/19068#discussion_r2029114746 ## clients/clients-integration-tests/src/test/java/org/apache/kafka/server/quota/CustomQuotaCallbackTest.java: ## @@ -121,4 +165,17 @@ public void configure(Map configs

Re: [PR] KAFKA-18754: Allow `PlainSaslServer` to pass client info through the negotiated properties. [kafka]

2025-04-04 Thread via GitHub
chia7712 commented on PR #18828: URL: https://github.com/apache/kafka/pull/18828#issuecomment-2765456093 > but we're facing an issue—PlainSaslServer can only pass the authorizationID to the authorizer, meaning the client info is lost. Have you considered implementing a custom `SaslSer

Re: [PR] KAFKA-14523: Move DelayedRemoteListOffsets to the storage module [kafka]

2025-04-04 Thread via GitHub
mimaison commented on code in PR #19285: URL: https://github.com/apache/kafka/pull/19285#discussion_r2021190137 ## storage/src/main/java/org/apache/kafka/server/purgatory/DelayedRemoteListOffsets.java: ## @@ -0,0 +1,198 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] KAFKA-18991: FetcherThread should match leader epochs between fetch request and fetch state [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on code in PR #19223: URL: https://github.com/apache/kafka/pull/19223#discussion_r2008619688 ## core/src/test/scala/unit/kafka/server/AbstractFetcherThreadTest.scala: ## @@ -1153,4 +1153,29 @@ class AbstractFetcherThreadTest { assertTrue(fetcher.fetchSt

Re: [PR] KAFKA-18999: Remove BrokerMetadata [kafka]

2025-04-04 Thread via GitHub
FrankYang0529 commented on code in PR #19227: URL: https://github.com/apache/kafka/pull/19227#discussion_r2007370090 ## core/src/main/scala/kafka/server/metadata/KRaftMetadataCache.scala: ## @@ -341,10 +341,10 @@ class KRaftMetadataCache( Option(_currentImage.cluster.broke

Re: [PR] KAFKA 18238 - fIdentify the first commit where a test became flaky [kafka]

2025-04-04 Thread via GitHub
github-actions[bot] commented on PR #18334: URL: https://github.com/apache/kafka/pull/18334#issuecomment-2777485586 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] [Updated] (KAFKA-19022) Display cluster IDs being compared when encountering INCONSISTENT_CLUSTER_ID error

2025-04-04 Thread Ranganath Samudrala (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19022?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ranganath Samudrala updated KAFKA-19022: Description: While migrating Kafka from zookeeper to kraft, we see errors in logs

Re: [PR] KAFKA-18067: Add a flag to disable producer reset during active task creator shutting down [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on PR #19269: URL: https://github.com/apache/kafka/pull/19269#issuecomment-2746260805 Hi @ableegoldman This is a reimplementation of 18067. Please take a look when you have a free moment. Many thanks 🙇🏼 -- This is an automated message from the Apache Git Ser

[PR] Another change but this time with a long title. What happens if the title is over 72 characters? I guess we'll find out. [kafka-merge-queue-sandbox]

2025-04-04 Thread via GitHub
mumrah opened a new pull request, #66: URL: https://github.com/apache/kafka-merge-queue-sandbox/pull/66 Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco labo

Re: [PR] KAFKA-19013 Reformat PR body to 72 characters [kafka]

2025-04-04 Thread via GitHub
mumrah commented on PR #19242: URL: https://github.com/apache/kafka/pull/19242#issuecomment-2743749389 @ijuma I don't think GH will truncate the commit subject, but I'm not sure -- I'll try it out on the sandbox repo. It is _really_ annoying that we can't tweak the commit message with

Re: [PR] KAFKA-14523: Move DelayedRemoteListOffsets to the storage module [kafka]

2025-04-04 Thread via GitHub
mimaison commented on code in PR #19285: URL: https://github.com/apache/kafka/pull/19285#discussion_r2029279945 ## storage/src/main/java/org/apache/kafka/server/purgatory/DelayedRemoteListOffsets.java: ## @@ -0,0 +1,200 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[jira] [Commented] (KAFKA-19012) Messages ending up on the wrong topic

2025-04-04 Thread Donny Nadolny (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17939732#comment-17939732 ] Donny Nadolny commented on KAFKA-19012: --- > I know it's a long shot, but could we p

Re: [PR] KAFKA-16729: Support isolation level for share consumer [kafka]

2025-04-04 Thread via GitHub
AndrewJSchofield commented on code in PR #19261: URL: https://github.com/apache/kafka/pull/19261#discussion_r2025141887 ## core/src/main/scala/kafka/server/KafkaApis.scala: ## @@ -3199,13 +3199,21 @@ class KafkaApis(val requestChannel: RequestChannel, request.context.

[PR] KAFKA-19064: Handle exceptions from deferred events in coordinator [kafka]

2025-04-04 Thread via GitHub
squah-confluent opened a new pull request, #19333: URL: https://github.com/apache/kafka/pull/19333 Previously in KAFKA-18484, we added exception handling for exceptions coming from batch events. Also handle exceptions from 0-record write events and transaction completion events. -

Re: [PR] MINOR: Cleanup Raft Module [kafka]

2025-04-04 Thread via GitHub
chia7712 commented on code in PR #19284: URL: https://github.com/apache/kafka/pull/19284#discussion_r2022952520 ## raft/src/main/java/org/apache/kafka/raft/ControlRecord.java: ## @@ -80,20 +80,14 @@ public ControlRecordType type() { } public short version() { Review

Re: [PR] Minor: Add functionalinterface to the producer callback [kafka]

2025-04-04 Thread via GitHub
Tombert commented on code in PR #19366: URL: https://github.com/apache/kafka/pull/19366#discussion_r2027527786 ## clients/src/main/java/org/apache/kafka/clients/consumer/AcknowledgementCommitCallback.java: ## @@ -34,6 +34,7 @@ * The callback may be executed in any thread calli

Re: [PR] update with reentrant lock [kafka]

2025-04-04 Thread via GitHub
github-actions[bot] commented on PR #17978: URL: https://github.com/apache/kafka/pull/17978#issuecomment-2763069973 This PR has been closed since it has not had any activity in 120 days. If you feel like this was a mistake, or you would like to continue working on it, please feel free to

Re: [PR] KAFKA-18892: KIP-877 Add support for ClientQuotaCallback [kafka]

2025-04-04 Thread via GitHub
m1a2st commented on PR #19068: URL: https://github.com/apache/kafka/pull/19068#issuecomment-2766450546 Thanks for @mimaison review, all comments addressed. -- 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

Re: [PR] [WIP] JWT bearer grant type support [kafka]

2025-04-04 Thread via GitHub
github-actions[bot] commented on PR #18912: URL: https://github.com/apache/kafka/pull/18912#issuecomment-2767974798 A label of 'needs-attention' was automatically added to this PR in order to raise the attention of the committers. Once this issue has been triaged, the `triage` label s

Re: [PR] [WIP] KAFKA-14830: Illegal state error in transactional producer [kafka]

2025-04-04 Thread via GitHub
kirktrue commented on PR #17022: URL: https://github.com/apache/kafka/pull/17022#issuecomment-2767367218 > I think the overall approach makes sense. But I would like to see some tests to see if the issue is improved. If so the logging could also give us some more insight. The unit te

Re: [PR] KAFKA-18899: Improve handling of timeouts for commitAsync() in ShareConsumer. [kafka]

2025-04-04 Thread via GitHub
ShivsundarR commented on code in PR #19192: URL: https://github.com/apache/kafka/pull/19192#discussion_r2012626389 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/ShareConsumeRequestManagerTest.java: ## @@ -1223,21 +1343,15 @@ public void testAcknowledgemen

Re: [PR] KAFKA-16729: Support isolation level for share consumer [kafka]

2025-04-04 Thread via GitHub
junrao commented on code in PR #19261: URL: https://github.com/apache/kafka/pull/19261#discussion_r2029290363 ## core/src/main/java/kafka/server/share/SharePartition.java: ## @@ -2484,6 +2505,182 @@ private long startOffsetDuringInitialization(long partitionDataStartOffset) thr

[PR] KAFKA-18796: Added more information to error message when assertion fails for acquisition lock timeout [kafka]

2025-04-04 Thread via GitHub
chirag-wadhwa5 opened a new pull request, #19247: URL: https://github.com/apache/kafka/pull/19247 This PR adds extra information in assertion failed messages for tests in SharePartitionTest revolving around acquisition lock timeouts functionality. -- This is an automated message from the

[jira] [Commented] (KAFKA-19065) Cleanup canUseTopicIds

2025-04-04 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940056#comment-17940056 ] Chia-Ping Tsai commented on KAFKA-19065: this ticket should include https://git

Re: [PR] MINOR: Run CI with Java 24 and update documentation (instead of 23) [kafka]

2025-04-04 Thread via GitHub
frankvicky commented on PR #19235: URL: https://github.com/apache/kafka/pull/19235#issuecomment-2735304675 It seems the gradle team is still working on it. https://github.com/gradle/gradle/pull/32576 -- This is an automated message from the Apache Git Service. To respond to the message,

[jira] [Resolved] (KAFKA-19067) AsyncKafkaConsumer may return stale fetch result after seek operation

2025-04-04 Thread PoAn Yang (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19067?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] PoAn Yang resolved KAFKA-19067. --- Resolution: Won't Fix > AsyncKafkaConsumer may return stale fetch result after seek operation >

Re: [PR] KAFKA-18891: KIP-877 add support for RemoteLogMetadataManager and RemoteStorageManager [kafka]

2025-04-04 Thread via GitHub
TaiJuWu commented on code in PR #19286: URL: https://github.com/apache/kafka/pull/19286#discussion_r2027851831 ## core/src/test/java/kafka/log/remote/RemoteLogManagerTest.java: ## @@ -413,24 +418,19 @@ void testRemoteLogMetadataManagerWithEndpointConfigOverridden() throws IOExc

[jira] [Commented] (KAFKA-18042) Reject the produce request with lower producer epoch early.

2025-04-04 Thread Justine Olshan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18042?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940838#comment-17940838 ] Justine Olshan commented on KAFKA-18042: Hey [~taijuwu] if you would like to wor

<    1   2   3   >