Re: [PR] KAFKA-19264: Remove fallback for thread pool sizes in RemoteLogManagerConfig [kafka]

2025-05-11 Thread via GitHub
kamalcph commented on PR #19673: URL: https://github.com/apache/kafka/pull/19673#issuecomment-2869962173 LGTM, thanks for fixing this! -- 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: Move TxnTransitMetadata to transaction-coordinator [kafka]

2025-05-11 Thread via GitHub
chia7712 merged PR #19662: URL: https://github.com/apache/kafka/pull/19662 -- 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] [Commented] (KAFKA-19267) the min version used by ListOffsetsRequest should be 1 rather than 0

2025-05-11 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19267?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17950784#comment-17950784 ] Chia-Ping Tsai commented on KAFKA-19267: related code: https://github.com/apach

[jira] [Created] (KAFKA-19267) the min version used by ListOffsetsRequest should be 1 rather than 0

2025-05-11 Thread Chia-Ping Tsai (Jira)
Chia-Ping Tsai created KAFKA-19267: -- Summary: the min version used by ListOffsetsRequest should be 1 rather than 0 Key: KAFKA-19267 URL: https://issues.apache.org/jira/browse/KAFKA-19267 Project: Kaf

[jira] [Assigned] (KAFKA-19267) the min version used by ListOffsetsRequest should be 1 rather than 0

2025-05-11 Thread Yunchi Pang (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19267?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yunchi Pang reassigned KAFKA-19267: --- Assignee: Yunchi Pang (was: Chia-Ping Tsai) > the min version used by ListOffsetsRequest s

Re: [PR] MINOR: Fix unchecked type warnings in several test classes [kafka]

2025-05-11 Thread via GitHub
frankvicky commented on code in PR #19679: URL: https://github.com/apache/kafka/pull/19679#discussion_r2083898755 ## metadata/src/test/java/org/apache/kafka/image/TopicsImageTest.java: ## @@ -446,7 +446,7 @@ public void testClearElrRecordForNonExistTopic() { @Test pu

Re: [PR] KAFKA-19253: Improve metadata handling for share version using feature listeners (1/N) [kafka]

2025-05-11 Thread via GitHub
AndrewJSchofield commented on code in PR #19659: URL: https://github.com/apache/kafka/pull/19659#discussion_r2083611522 ## core/src/main/java/kafka/server/share/SharePartitionManager.java: ## @@ -746,6 +747,29 @@ private Consumer> failedShareAcknowledgeMetricsHandler() {

[PR] MINOR: Minor updates to RangeSet [kafka]

2025-05-11 Thread via GitHub
squah-confluent opened a new pull request, #19678: URL: https://github.com/apache/kafka/pull/19678 Minor updates to RangeSet: * Fix size() to return 0 when to < from * Fix equals() so that all empty RangeSets are equal, to follow the Set interface definition better. * Reimplement

Re: [PR] MINOR: fit HTML markup [kafka]

2025-05-11 Thread via GitHub
mjsax merged PR #19676: URL: https://github.com/apache/kafka/pull/19676 -- 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] MINOR: Fix unchecked type warnings in several test classes [kafka]

2025-05-11 Thread via GitHub
m1a2st commented on code in PR #19679: URL: https://github.com/apache/kafka/pull/19679#discussion_r2083766600 ## metadata/src/test/java/org/apache/kafka/image/TopicsImageTest.java: ## @@ -446,7 +446,7 @@ public void testClearElrRecordForNonExistTopic() { @Test public

Re: [PR] KAFKA-17828: Reverse Checkpointing in MM2 [kafka]

2025-05-11 Thread via GitHub
github-actions[bot] commented on PR #17593: URL: https://github.com/apache/kafka/pull/17593#issuecomment-2870720484 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

Re: [PR] MINOR: Fix unchecked type warnings in several test classes [kafka]

2025-05-11 Thread via GitHub
YutaLin commented on code in PR #19679: URL: https://github.com/apache/kafka/pull/19679#discussion_r2083773873 ## metadata/src/test/java/org/apache/kafka/image/TopicsImageTest.java: ## @@ -446,7 +446,7 @@ public void testClearElrRecordForNonExistTopic() { @Test publi

Re: [PR] MINOR: Fix unchecked type warnings in several test classes [kafka]

2025-05-11 Thread via GitHub
m1a2st commented on code in PR #19679: URL: https://github.com/apache/kafka/pull/19679#discussion_r2083777217 ## metadata/src/test/java/org/apache/kafka/image/TopicsImageTest.java: ## @@ -446,7 +446,7 @@ public void testClearElrRecordForNonExistTopic() { @Test public

[jira] [Created] (KAFKA-19266) Eliminate flakiness in test SharePartitionTest.testAcquisitionLockTimeoutForBatchesPostStartOffsetMovement

2025-05-11 Thread Abhinav Dixit (Jira)
Abhinav Dixit created KAFKA-19266: - Summary: Eliminate flakiness in test SharePartitionTest.testAcquisitionLockTimeoutForBatchesPostStartOffsetMovement Key: KAFKA-19266 URL: https://issues.apache.org/jira/browse/K

Re: [PR] KAFKA-16717 [2/N]: Add AdminClient.alterShareGroupOffsets [kafka]

2025-05-11 Thread via GitHub
AndrewJSchofield commented on code in PR #18929: URL: https://github.com/apache/kafka/pull/18929#discussion_r2083622116 ## core/src/main/scala/kafka/server/KafkaApis.scala: ## @@ -3620,7 +3620,50 @@ class KafkaApis(val requestChannel: RequestChannel, def handleAlterShareGro

Re: [PR] MINOR: Upgrade jetty version [kafka]

2025-05-11 Thread via GitHub
frankvicky merged PR #19680: URL: https://github.com/apache/kafka/pull/19680 -- 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.apac

Re: [PR] MINOR: Fix unchecked type warnings in several test classes [kafka]

2025-05-11 Thread via GitHub
frankvicky commented on code in PR #19679: URL: https://github.com/apache/kafka/pull/19679#discussion_r2083748605 ## metadata/src/test/java/org/apache/kafka/image/TopicsImageTest.java: ## @@ -446,7 +446,7 @@ public void testClearElrRecordForNonExistTopic() { @Test pu

Re: [PR] Refactored DescribeTopicPartitionsRequestHandler: Improve readability and add code documentation [kafka]

2025-05-11 Thread via GitHub
github-actions[bot] commented on PR #19636: URL: https://github.com/apache/kafka/pull/19636#issuecomment-2870683846 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] KAFKA-19091: Fix race condition in DelayedFutureTest [kafka]

2025-05-11 Thread via GitHub
github-actions[bot] commented on PR #19553: URL: https://github.com/apache/kafka/pull/19553#issuecomment-2870683892 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] KAFKA-18066: Fix mismatched StreamThread ID in log messages [kafka]

2025-05-11 Thread via GitHub
github-actions[bot] commented on PR #19517: URL: https://github.com/apache/kafka/pull/19517#issuecomment-2870683922 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] KAFKA-17747: [2/N] Add compute topic and group hash [kafka]

2025-05-11 Thread via GitHub
ijuma commented on code in PR #19523: URL: https://github.com/apache/kafka/pull/19523#discussion_r2083606229 ## group-coordinator/src/test/java/org/apache/kafka/coordinator/group/UtilsTest.java: ## @@ -0,0 +1,201 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] KAFKA-17747: [2/N] Add compute topic and group hash [kafka]

2025-05-11 Thread via GitHub
ijuma commented on code in PR #19523: URL: https://github.com/apache/kafka/pull/19523#discussion_r2083606336 ## group-coordinator/src/test/java/org/apache/kafka/coordinator/group/UtilsTest.java: ## @@ -0,0 +1,201 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[jira] [Resolved] (KAFKA-18695) Remove quorum=kraft and kip932 from all integration tests

2025-05-11 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18695?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai resolved KAFKA-18695. Fix Version/s: 4.1.0 Resolution: Fixed > Remove quorum=kraft and kip932 from all in

[jira] [Updated] (KAFKA-18695) Remove quorum=kraft and kip932 from all integration tests

2025-05-11 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-18695?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai updated KAFKA-18695: --- Summary: Remove quorum=kraft and kip932 from all integration tests (was: Remove quorum=kraf

Re: [PR] KAFKA-18695: Remove quorum=kraft and kip932 from all integration tests [kafka]

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

[PR] MINOR: fit HTML markup [kafka]

2025-05-11 Thread via GitHub
mjsax opened a new pull request, #19676: URL: https://github.com/apache/kafka/pull/19676 (no comment) -- 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

Re: [PR] KAFKA-18904: [1/N] Change ListClientMetricsResources API to ListConfigResources [kafka]

2025-05-11 Thread via GitHub
AndrewJSchofield commented on code in PR #19493: URL: https://github.com/apache/kafka/pull/19493#discussion_r2083608078 ## core/src/test/scala/unit/kafka/server/KafkaApisTest.scala: ## @@ -10929,47 +10928,231 @@ class KafkaApisTest extends Logging { } @Test - def testLi

Re: [PR] KAFKA-19264: Remove fallback for thread pool sizes in RemoteLogManagerConfig [kafka]

2025-05-11 Thread via GitHub
chia7712 merged PR #19673: URL: https://github.com/apache/kafka/pull/19673 -- 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] [Resolved] (KAFKA-19264) Remove fallback for thread pool sizes in RemoteLogManagerConfig

2025-05-11 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chia-Ping Tsai resolved KAFKA-19264. Fix Version/s: 4.1.0 Resolution: Fixed > Remove fallback for thread pool sizes in R

[PR] MINOR: Add deprecation warning for `log.cleaner.enable` when disabled [kafka]

2025-05-11 Thread via GitHub
Mirai1129 opened a new pull request, #19674: URL: https://github.com/apache/kafka/pull/19674 This parameter `log.cleaner.enable` has been deprecated, in Kafka 5.0 later cannot disable cleaner. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] MINOR: Add deprecation warning for `log.cleaner.enable` when disabled [kafka]

2025-05-11 Thread via GitHub
chia7712 commented on code in PR #19674: URL: https://github.com/apache/kafka/pull/19674#discussion_r2083581944 ## core/src/main/scala/kafka/log/LogManager.scala: ## @@ -629,6 +629,8 @@ class LogManager(logDirs: Seq[File], if (cleanerConfig.enableCleaner) { _cleaner

Re: [PR] MINOR: Add deprecation warning for `log.cleaner.enable` when disabled [kafka]

2025-05-11 Thread via GitHub
Mirai1129 commented on code in PR #19674: URL: https://github.com/apache/kafka/pull/19674#discussion_r2083583209 ## core/src/main/scala/kafka/log/LogManager.scala: ## @@ -629,6 +629,8 @@ class LogManager(logDirs: Seq[File], if (cleanerConfig.enableCleaner) { _cleaner

Re: [PR] KAFKA-18688: Fix uniform homogeneous assignor stability [kafka]

2025-05-11 Thread via GitHub
squah-confluent commented on PR #19677: URL: https://github.com/apache/kafka/pull/19677#issuecomment-2870221833 Benchmarks before and after Scroll right for after numbers. ``` Benchmark: ServerSideAssignorBenchmark.doAssignment assignorType: UNIFORM isRackAware: fal

[PR] MINOR: Upgrade jetty version [kafka]

2025-05-11 Thread via GitHub
frankvicky opened a new pull request, #19680: URL: https://github.com/apache/kafka/pull/19680 When building RC, the current version of Jetty has been reporting for CVE. Hence, we should upgrade the Jetty version to fix it. -- This is an automated message from the Apache Git Ser

[jira] [Created] (KAFKA-19264) Remove fallback for thread pool sizes in RemoteLogManagerConfig

2025-05-11 Thread Kuan Po Tseng (Jira)
Kuan Po Tseng created KAFKA-19264: - Summary: Remove fallback for thread pool sizes in RemoteLogManagerConfig Key: KAFKA-19264 URL: https://issues.apache.org/jira/browse/KAFKA-19264 Project: Kafka

[jira] [Updated] (KAFKA-19264) Remove fallback for thread pool sizes in RemoteLogManagerConfig

2025-05-11 Thread Kuan Po Tseng (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kuan Po Tseng updated KAFKA-19264: -- Description: The fallback mechanism was first introduced in [KIP-950|https://cwiki.apache.org

[jira] [Resolved] (KAFKA-19139) Plugin#wrapInstance should use LinkedHashMap instead of Map

2025-05-11 Thread TaiJuWu (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] TaiJuWu resolved KAFKA-19139. - Resolution: Fixed > Plugin#wrapInstance should use LinkedHashMap instead of Map > -

[PR] KAFKA-19264: Remove fallback for thread pool sizes in RemoteLogManagerConfig [kafka]

2025-05-11 Thread via GitHub
brandboat opened a new pull request, #19673: URL: https://github.com/apache/kafka/pull/19673 The fallback mechanism for `remote.log.manager.copier.thread.pool.size` and `remote.log.manager.expiration.thread.pool.size` defaulting to `remote.log.manager.thread.pool.size` was introduced in KIP

[jira] [Updated] (KAFKA-19265) Use snapshotting of remote log metedata topic that is used when a broker is restarted.

2025-05-11 Thread Satish Duggana (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19265?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Satish Duggana updated KAFKA-19265: --- Description: This is an incremental approach to the remote log metadata snapshot mechanism a

[jira] [Created] (KAFKA-19265) Use snapshotting of remote log metedata topic that is used when a broker is restarted.

2025-05-11 Thread Satish Duggana (Jira)
Satish Duggana created KAFKA-19265: -- Summary: Use snapshotting of remote log metedata topic that is used when a broker is restarted. Key: KAFKA-19265 URL: https://issues.apache.org/jira/browse/KAFKA-19265

[jira] [Commented] (KAFKA-13610) Make log.cleaner.enable dynamically configurable

2025-05-11 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17950756#comment-17950756 ] Chia-Ping Tsai commented on KAFKA-13610: We should change the min value of log.c

[jira] [Created] (KAFKA-19262) Add a test to verify all metrics naming pattern

2025-05-11 Thread Jira
黃竣陽 created KAFKA-19262: --- Summary: Add a test to verify all metrics naming pattern Key: KAFKA-19262 URL: https://issues.apache.org/jira/browse/KAFKA-19262 Project: Kafka Issue Type: Improvement

[jira] [Assigned] (KAFKA-19263) The docs of delete.topic.enable used by Admin#deletetopics is out-of-date

2025-05-11 Thread Yu Chia Ma (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-19263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yu Chia Ma reassigned KAFKA-19263: -- Assignee: Yu Chia Ma > The docs of delete.topic.enable used by Admin#deletetopics is out-of-

[jira] [Created] (KAFKA-19263) The docs of delete.topic.enable used by Admin#deletetopics is out-of-date

2025-05-11 Thread Chia-Ping Tsai (Jira)
Chia-Ping Tsai created KAFKA-19263: -- Summary: The docs of delete.topic.enable used by Admin#deletetopics is out-of-date Key: KAFKA-19263 URL: https://issues.apache.org/jira/browse/KAFKA-19263 Projec

Re: [PR] KAFKA-19242: Fix commit bugs caused by race condition during rebalancing. [kafka]

2025-05-11 Thread via GitHub
chickenchickenlove commented on PR #19631: URL: https://github.com/apache/kafka/pull/19631#issuecomment-2869870166 @dajac Sorry to bother you...! Gently ping 🙇‍♂️ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[PR] KAFKA-19263: update `delete.topic.enable` docs [kafka]

2025-05-11 Thread via GitHub
Mirai1129 opened a new pull request, #19675: URL: https://github.com/apache/kafka/pull/19675 If delete.topic.enable is false on the brokers, deleteTopics will mark the topics for deletion, but not actually delete them. The futures will return successfully in this case. It is not true

[PR] MINOR: Fix unchecked type warnings in several test classes [kafka]

2025-05-11 Thread via GitHub
YutaLin opened a new pull request, #19679: URL: https://github.com/apache/kafka/pull/19679 * In ConsoleShareConsumerTest, add @SuppressWarnings("unchecked") annotation in method shouldUpgradeDeliveryCount * In ListConsumerGroupOffsetsHandlerTest, add generic parameters to HashSet constru

Re: [PR] KAFKA-14691; Add TopicId to OffsetFetch API [kafka]

2025-05-11 Thread via GitHub
frankvicky commented on code in PR #19515: URL: https://github.com/apache/kafka/pull/19515#discussion_r2083791909 ## clients/src/test/java/org/apache/kafka/common/requests/OffsetFetchRequestTest.java: ## @@ -42,17 +43,21 @@ public void testWithMultipleGroups(short version) {

Re: [PR] KAFKA-14691; Add TopicId to OffsetFetch API [kafka]

2025-05-11 Thread via GitHub
frankvicky commented on code in PR #19515: URL: https://github.com/apache/kafka/pull/19515#discussion_r2083796590 ## clients/src/test/java/org/apache/kafka/common/requests/OffsetFetchRequestTest.java: ## @@ -42,17 +43,21 @@ public void testWithMultipleGroups(short version) {