[jira] [Commented] (KAFKA-14920) Address timeouts and out of order sequences

2023-04-21 Thread Justine Olshan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715227#comment-17715227 ] Justine Olshan commented on KAFKA-14920: I originally considered this, but we can't do it

[GitHub] [kafka] dajac commented on pull request #13571: KAFKA-14907:Add the traffic metric of the partition dimension in BrokerTopicStats

2023-04-21 Thread via GitHub
dajac commented on PR #13571: URL: https://github.com/apache/kafka/pull/13571#issuecomment-1518511048 > Does this PR need a KIP process? Anyone take a look? Metrics are part of the public api. Changing or adding metrics require a KIP. -- This is an automated message from the Apache

[jira] [Commented] (KAFKA-14920) Address timeouts and out of order sequences

2023-04-21 Thread Artem Livshits (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715224#comment-17715224 ] Artem Livshits commented on KAFKA-14920: I wonder if we could just add producer state before

[GitHub] [kafka] artemlivshits commented on a diff in pull request #13267: KAFKA-14694: RPCProducerIdManager should not wait on new block

2023-04-21 Thread via GitHub
artemlivshits commented on code in PR #13267: URL: https://github.com/apache/kafka/pull/13267#discussion_r1174283192 ## core/src/main/scala/kafka/coordinator/transaction/ProducerIdManager.scala: ## @@ -123,73 +129,94 @@ class ZkProducerIdManager(brokerId: Int, } } -

[jira] [Commented] (KAFKA-14925) The website shouldn't load external resources

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715205#comment-17715205 ] ASF GitHub Bot commented on KAFKA-14925: atu-sharm commented on PR #506: URL:

[jira] [Commented] (KAFKA-14925) The website shouldn't load external resources

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715204#comment-17715204 ] ASF GitHub Bot commented on KAFKA-14925: atu-sharm opened a new pull request, #506: URL:

[jira] [Commented] (KAFKA-14892) Harmonize package names in storage module

2023-04-21 Thread Jun Rao (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14892?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715193#comment-17715193 ] Jun Rao commented on KAFKA-14892: - Sounds good to me. Including storage in the package name probably

[GitHub] [kafka] bmscomp opened a new pull request, #13625: MINOR: Upgrade gradle wrapper to the last stable version 8.1.1

2023-04-21 Thread via GitHub
bmscomp opened a new pull request, #13625: URL: https://github.com/apache/kafka/pull/13625 Upgrade Gradle wrapper to the latest stable version 8.1.1 and update the value of property name `distributionSha256Sum` ### Committer Checklist (excluded from commit message) - [x] Verify

[jira] [Commented] (KAFKA-8115) Flaky Test CoordinatorTest#testTaskRequestWithOldStartMsGetsUpdated

2023-04-21 Thread Greg Harris (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-8115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715177#comment-17715177 ] Greg Harris commented on KAFKA-8115: We merged an attempted fix for this, but it appears that

[GitHub] [kafka] gharris1727 commented on pull request #13557: KAFKA-14902: KafkaStatusBackingStore retries on a dedicated backgroun…

2023-04-21 Thread via GitHub
gharris1727 commented on PR #13557: URL: https://github.com/apache/kafka/pull/13557#issuecomment-1518351859 Looks like this might have caused some flaky failures on trunk: https://issues.apache.org/jira/browse/KAFKA-14929 -- This is an automated message from the Apache Git Service. To

[jira] [Updated] (KAFKA-14929) Flaky KafkaStatusBackingStoreFormatTest#putTopicStateRetriableFailure

2023-04-21 Thread Greg Harris (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Greg Harris updated KAFKA-14929: Labels: flaky-test (was: ) > Flaky

[jira] [Updated] (KAFKA-14929) Flaky KafkaStatusBackingStoreFormatTest#putTopicStateRetriableFailure

2023-04-21 Thread Greg Harris (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Greg Harris updated KAFKA-14929: Description: This test recently started flaky-failing with the following stack trace: {noformat}

[GitHub] [kafka] philipnee commented on a diff in pull request #13490: KAFKA-14875: Implement wakeup

2023-04-21 Thread via GitHub
philipnee commented on code in PR #13490: URL: https://github.com/apache/kafka/pull/13490#discussion_r1174174040 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/PrototypeAsyncConsumer.java: ## @@ -250,6 +257,7 @@ public void commitAsync(OffsetCommitCallback

[jira] [Created] (KAFKA-14929) Flaky KafkaStatusBackingStoreFormatTest#putTopicStateRetriableFailure

2023-04-21 Thread Greg Harris (Jira)
Greg Harris created KAFKA-14929: --- Summary: Flaky KafkaStatusBackingStoreFormatTest#putTopicStateRetriableFailure Key: KAFKA-14929 URL: https://issues.apache.org/jira/browse/KAFKA-14929 Project: Kafka

[GitHub] [kafka] hachikuji commented on a diff in pull request #13490: KAFKA-14875: Implement wakeup

2023-04-21 Thread via GitHub
hachikuji commented on code in PR #13490: URL: https://github.com/apache/kafka/pull/13490#discussion_r1169316955 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/PrototypeAsyncConsumer.java: ## @@ -250,6 +257,7 @@ public void commitAsync(OffsetCommitCallback

[jira] [Updated] (KAFKA-14905) Failing tests in MM2 ForwardingAdmin test since KIP-894

2023-04-21 Thread Josep Prat (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14905?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Josep Prat updated KAFKA-14905: --- Fix Version/s: 3.4.1 3.6.0 > Failing tests in MM2 ForwardingAdmin test since

[GitHub] [kafka] hachikuji commented on pull request #13551: MINOR: Allow tagged fields with version subset of flexible version range

2023-04-21 Thread via GitHub
hachikuji commented on PR #13551: URL: https://github.com/apache/kafka/pull/13551#issuecomment-1518292028 @cmccabe I think that case is already covered here: https://github.com/apache/kafka/blob/trunk/generator/src/main/java/org/apache/kafka/message/MessageDataGenerator.java#L425. The case

[GitHub] [kafka] jlprat merged pull request #13575: KAFKA-14905: Reduce flakiness in MM2 ForwardingAdmin test due to admin timeouts

2023-04-21 Thread via GitHub
jlprat merged PR #13575: URL: https://github.com/apache/kafka/pull/13575 -- 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:

[GitHub] [kafka] jlprat commented on pull request #13575: KAFKA-14905: Reduce flakiness in MM2 ForwardingAdmin test due to admin timeouts

2023-04-21 Thread via GitHub
jlprat commented on PR #13575: URL: https://github.com/apache/kafka/pull/13575#issuecomment-1518270904 Yes, I was looking at the `rat` as well, one but I fail to see how they would be caused in any way by the changes you made. I seee

[GitHub] [kafka] mjsax commented on pull request #13577: MINOR: updated KS release notes for 3.5

2023-04-21 Thread via GitHub
mjsax commented on PR #13577: URL: https://github.com/apache/kafka/pull/13577#issuecomment-1518267444 Merged to `trunk` and cherry-picked to `3.5`. -- 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] [Commented] (KAFKA-14876) Public documentation for new Kafka Connect offset management REST APIs

2023-04-21 Thread Greg Harris (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715158#comment-17715158 ] Greg Harris commented on KAFKA-14876: - I think the stop API is relevant to document, even without

[GitHub] [kafka] mjsax merged pull request #13577: MINOR: updated KS release notes for 3.5

2023-04-21 Thread via GitHub
mjsax merged PR #13577: URL: https://github.com/apache/kafka/pull/13577 -- 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] [Commented] (KAFKA-14722) Make BooleanSerde public

2023-04-21 Thread Matthias J. Sax (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715157#comment-17715157 ] Matthias J. Sax commented on KAFKA-14722: - I did a PR:

[GitHub] [kafka] gharris1727 commented on pull request #13575: KAFKA-14905: Reduce flakiness in MM2 ForwardingAdmin test due to admin timeouts

2023-04-21 Thread via GitHub
gharris1727 commented on PR #13575: URL: https://github.com/apache/kafka/pull/13575#issuecomment-1518264915 hmm some build failures from `rat` like in #13315 but I don't think this PR is related. And the JDK17 build failed due to streams upgrade tests. -- This is an automated message

[GitHub] [kafka] jlprat commented on a diff in pull request #13575: KAFKA-14905: Reduce flakiness in MM2 ForwardingAdmin test due to admin timeouts

2023-04-21 Thread via GitHub
jlprat commented on code in PR #13575: URL: https://github.com/apache/kafka/pull/13575#discussion_r1174097520 ## connect/mirror/src/test/java/org/apache/kafka/connect/mirror/clients/admin/FakeForwardingAdminWithLocalMetadata.java: ## @@ -38,16 +38,13 @@ import

[GitHub] [kafka] szalapski commented on pull request #5876: KAFKA-7509: Avoid passing most non-applicable properties to producer, consumer, and admin client

2023-04-21 Thread via GitHub
szalapski commented on PR #5876: URL: https://github.com/apache/kafka/pull/5876#issuecomment-1518174551 @rhauch Any idea how to move this along? -- 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

[GitHub] [kafka] gharris1727 commented on a diff in pull request #13429: KAFKA-14666: Add MM2 in-memory offset translation index for offsets behind replication

2023-04-21 Thread via GitHub
gharris1727 commented on code in PR #13429: URL: https://github.com/apache/kafka/pull/13429#discussion_r1173998391 ## connect/mirror/src/main/java/org/apache/kafka/connect/mirror/OffsetSyncStore.java: ## @@ -139,10 +171,103 @@ public void close() { protected void

[GitHub] [kafka] gharris1727 commented on a diff in pull request #13429: KAFKA-14666: Add MM2 in-memory offset translation index for offsets behind replication

2023-04-21 Thread via GitHub
gharris1727 commented on code in PR #13429: URL: https://github.com/apache/kafka/pull/13429#discussion_r1173998391 ## connect/mirror/src/main/java/org/apache/kafka/connect/mirror/OffsetSyncStore.java: ## @@ -139,10 +171,103 @@ public void close() { protected void

[GitHub] [kafka] gharris1727 commented on a diff in pull request #13429: KAFKA-14666: Add MM2 in-memory offset translation index for offsets behind replication

2023-04-21 Thread via GitHub
gharris1727 commented on code in PR #13429: URL: https://github.com/apache/kafka/pull/13429#discussion_r1173998391 ## connect/mirror/src/main/java/org/apache/kafka/connect/mirror/OffsetSyncStore.java: ## @@ -139,10 +171,103 @@ public void close() { protected void

[GitHub] [kafka] gharris1727 commented on a diff in pull request #13429: KAFKA-14666: Add MM2 in-memory offset translation index for offsets behind replication

2023-04-21 Thread via GitHub
gharris1727 commented on code in PR #13429: URL: https://github.com/apache/kafka/pull/13429#discussion_r1173998391 ## connect/mirror/src/main/java/org/apache/kafka/connect/mirror/OffsetSyncStore.java: ## @@ -139,10 +171,103 @@ public void close() { protected void

[jira] [Updated] (KAFKA-14666) MM2 should translate consumer group offsets behind replication flow

2023-04-21 Thread Greg Harris (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14666?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Greg Harris updated KAFKA-14666: Fix Version/s: 3.5.0 > MM2 should translate consumer group offsets behind replication flow >

[jira] [Updated] (KAFKA-14666) MM2 should translate consumer group offsets behind replication flow

2023-04-21 Thread Greg Harris (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14666?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Greg Harris updated KAFKA-14666: Affects Version/s: (was: 3.5.0) > MM2 should translate consumer group offsets behind

[GitHub] [kafka] gharris1727 commented on a diff in pull request #13575: KAFKA-14905: Reduce flakiness in MM2 ForwardingAdmin test due to admin timeouts

2023-04-21 Thread via GitHub
gharris1727 commented on code in PR #13575: URL: https://github.com/apache/kafka/pull/13575#discussion_r1173979163 ## connect/mirror/src/test/java/org/apache/kafka/connect/mirror/clients/admin/FakeForwardingAdminWithLocalMetadata.java: ## @@ -38,16 +38,13 @@ import

[GitHub] [kafka] dajac commented on pull request #13602: KAFKA-14869: Bump coordinator value records to flexible versions (KIP…

2023-04-21 Thread via GitHub
dajac commented on PR #13602: URL: https://github.com/apache/kafka/pull/13602#issuecomment-1518070622 @jeffkbkim Could you please rebase this one? -- 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

[GitHub] [kafka] dajac merged pull request #13598: KAFKA-14869: Ignore unknown record types for coordinators (KIP-915, P…

2023-04-21 Thread via GitHub
dajac merged PR #13598: URL: https://github.com/apache/kafka/pull/13598 -- 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:

[GitHub] [kafka] dajac commented on pull request #13598: KAFKA-14869: Ignore unknown record types for coordinators (KIP-915, P…

2023-04-21 Thread via GitHub
dajac commented on PR #13598: URL: https://github.com/apache/kafka/pull/13598#issuecomment-1518069951 I could not get a successful build because there is always an `Timeout waiting to lock zinc-1.3.5_2.12.15_8 compiler cache` error. However, two builds combines give us a good signal: *

[jira] [Updated] (KAFKA-14720) KIP-906: Tools migration guidelines

2023-04-21 Thread Federico Valeri (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14720?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Federico Valeri updated KAFKA-14720: Fix Version/s: 3.5.0 > KIP-906: Tools migration guidelines >

[jira] [Updated] (KAFKA-14633) Compression optimization: Use BufferSupplier to allocate the intermediate decompressed buffer

2023-04-21 Thread Divij Vaidya (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14633?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Divij Vaidya updated KAFKA-14633: - Attachment: flamegraph-pr-heapalloc-after.html

[GitHub] [kafka] ruslankrivoshein commented on pull request #13562: KAFKA-14581: Moving GetOffsetShell to tools

2023-04-21 Thread via GitHub
ruslankrivoshein commented on PR #13562: URL: https://github.com/apache/kafka/pull/13562#issuecomment-1518054816 Yes, since it's WIP and I have questions, I left original files. Well, I'll take a look at `ClusterTestExtensions` and will try to figure out more about testing there. --

[jira] [Commented] (KAFKA-10883) Properly finish the "get-started.html" page and link in in the top navigation menu

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715053#comment-17715053 ] ASF GitHub Bot commented on KAFKA-10883: machi1990 commented on PR #504: URL:

[jira] [Commented] (KAFKA-10883) Properly finish the "get-started.html" page and link in in the top navigation menu

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715054#comment-17715054 ] ASF GitHub Bot commented on KAFKA-10883: machi1990 closed pull request #504: KAFKA-10883: link

[GitHub] [kafka] rvansa commented on pull request #13619: Initial support for OpenJDK CRaC snapshotting

2023-04-21 Thread via GitHub
rvansa commented on PR #13619: URL: https://github.com/apache/kafka/pull/13619#issuecomment-1517992031 @Hangleton You're right, this process should be repeated; for example in the JDK itself we flush DNS caches before checkpoint. I was hoping that the code in

[jira] [Updated] (KAFKA-10883) Properly finish the "get-started.html" page and link in in the top navigation menu

2023-04-21 Thread Manyanda Chitimbo (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Manyanda Chitimbo updated KAFKA-10883: -- Description: The get-started page: [https://kafka.apache.org/get-started] currently

[jira] [Updated] (KAFKA-10883) Properly finish the "get-started.html" page and link in in the top navigation menu

2023-04-21 Thread Manyanda Chitimbo (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Manyanda Chitimbo updated KAFKA-10883: -- Description: The get-started page: [https://kafka.apache.org/get-started] currently

[jira] [Updated] (KAFKA-10883) Properly finish the "get-started.html" page and link in in the top navigation menu

2023-04-21 Thread Manyanda Chitimbo (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Manyanda Chitimbo updated KAFKA-10883: -- Description: The get-started page: [https://kafka.apache.org/get-started] currently

[jira] [Updated] (KAFKA-10883) Properly finish the "get-started.html" page and link in in the top navigation menu

2023-04-21 Thread Manyanda Chitimbo (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Manyanda Chitimbo updated KAFKA-10883: -- Summary: Properly finish the "get-started.html" page and link in in the top

[GitHub] [kafka] dajac commented on pull request #13603: KAFKA-14869: Bump coordinator value records to flexible versions (KIP…

2023-04-21 Thread via GitHub
dajac commented on PR #13603: URL: https://github.com/apache/kafka/pull/13603#issuecomment-1517975471 @jeffkbkim Could you please rebase this one? -- 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

[jira] [Commented] (KAFKA-12826) Remove Deprecated Class Serdes (Streams)

2023-04-21 Thread Josep Prat (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-12826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715047#comment-17715047 ] Josep Prat commented on KAFKA-12826: Unfortunately, this task can't be taken just yet. It can only

[GitHub] [kafka] dajac merged pull request #13599: KAFKA-14869: Ignore unknown record types for coordinators (KIP-915, P…

2023-04-21 Thread via GitHub
dajac merged PR #13599: URL: https://github.com/apache/kafka/pull/13599 -- 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] [Commented] (KAFKA-12826) Remove Deprecated Class Serdes (Streams)

2023-04-21 Thread shiqin.lan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-12826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715044#comment-17715044 ] shiqin.lan commented on KAFKA-12826: Class org.apache.kafka.streams.scala.Serdes was deprecated in

[jira] [Comment Edited] (KAFKA-14928) Metrics collection contends on lock with log cleaning

2023-04-21 Thread Alexandre Dupriez (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14928?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715041#comment-17715041 ] Alexandre Dupriez edited comment on KAFKA-14928 at 4/21/23 2:55 PM:

[jira] [Commented] (KAFKA-14928) Metrics collection contends on lock with log cleaning

2023-04-21 Thread Alexandre Dupriez (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14928?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715041#comment-17715041 ] Alexandre Dupriez commented on KAFKA-14928: --- Hi Divij, thanks for reporting this. Would you

[jira] [Commented] (KAFKA-10883) Lien hypertext incorrect

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715040#comment-17715040 ] ASF GitHub Bot commented on KAFKA-10883: mimaison commented on PR #504: URL:

[jira] [Updated] (KAFKA-14652) Improve MM2 logging by adding the flow information to the context (KIP-916)

2023-04-21 Thread Daniel Urban (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14652?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Urban updated KAFKA-14652: - Summary: Improve MM2 logging by adding the flow information to the context (KIP-916) (was:

[GitHub] [kafka] machi1990 commented on a diff in pull request #13623: KAFKA-14926: Remove metrics on Log Cleaner shutdown

2023-04-21 Thread via GitHub
machi1990 commented on code in PR #13623: URL: https://github.com/apache/kafka/pull/13623#discussion_r1173855980 ## core/src/main/scala/kafka/log/LogCleaner.scala: ## @@ -167,8 +167,20 @@ class LogCleaner(initialConfig: CleanerConfig, */ def shutdown(): Unit = {

[jira] [Commented] (KAFKA-10883) Lien hypertext incorrect

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715035#comment-17715035 ] ASF GitHub Bot commented on KAFKA-10883: machi1990 commented on PR #504: URL:

[jira] [Created] (KAFKA-14928) Metrics collection contends on lock with log cleaning

2023-04-21 Thread Divij Vaidya (Jira)
Divij Vaidya created KAFKA-14928: Summary: Metrics collection contends on lock with log cleaning Key: KAFKA-14928 URL: https://issues.apache.org/jira/browse/KAFKA-14928 Project: Kafka Issue

[jira] [Updated] (KAFKA-14927) Dynamic configs not validated when using kafka-configs and --add-config-file

2023-04-21 Thread Justin Daines (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14927?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justin Daines updated KAFKA-14927: -- Description: Using {{kafka-configs}} should validate dynamic configurations before applying.

[jira] [Created] (KAFKA-14927) Dynamic configs not validated when using kafka-configs and --add-config-file

2023-04-21 Thread Justin Daines (Jira)
Justin Daines created KAFKA-14927: - Summary: Dynamic configs not validated when using kafka-configs and --add-config-file Key: KAFKA-14927 URL: https://issues.apache.org/jira/browse/KAFKA-14927

[jira] [Commented] (KAFKA-14722) Make BooleanSerde public

2023-04-21 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715010#comment-17715010 ] Mickael Maison commented on KAFKA-14722: Have the docs updates been merged? Or is anyone working

[GitHub] [kafka] jlprat commented on a diff in pull request #13575: KAFKA-14905: Reduce flakiness in MM2 ForwardingAdmin test due to admin timeouts

2023-04-21 Thread via GitHub
jlprat commented on code in PR #13575: URL: https://github.com/apache/kafka/pull/13575#discussion_r1173773831 ## connect/mirror/src/test/java/org/apache/kafka/connect/mirror/clients/admin/FakeForwardingAdminWithLocalMetadata.java: ## @@ -38,16 +38,13 @@ import

[GitHub] [kafka] machi1990 commented on pull request #13624: MINOR: remove unused ProcessorNode#time field and remove unused imports

2023-04-21 Thread via GitHub
machi1990 commented on PR #13624: URL: https://github.com/apache/kafka/pull/13624#issuecomment-1517844646 Hi @guozhangwang @ableegoldman would you mind giving this a look when you've some time? Thanks -- This is an automated message from the Apache Git Service. To respond to the message,

[jira] [Commented] (KAFKA-10883) Lien hypertext incorrect

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715008#comment-17715008 ] ASF GitHub Bot commented on KAFKA-10883: mimaison commented on PR #504: URL:

[jira] [Commented] (KAFKA-10883) Lien hypertext incorrect

2023-04-21 Thread Manyanda Chitimbo (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17715005#comment-17715005 ] Manyanda Chitimbo commented on KAFKA-10883: --- I've opened a PR

[GitHub] [kafka] machi1990 commented on pull request #13612: MINOR: fix noticed typo in raft and metadata projects

2023-04-21 Thread via GitHub
machi1990 commented on PR #13612: URL: https://github.com/apache/kafka/pull/13612#issuecomment-1517827939 Thanks for the review @jlprat -- 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

[GitHub] [kafka] machi1990 opened a new pull request, #13624: MINOR: remove unused ProcessorNode#time field

2023-04-21 Thread via GitHub
machi1990 opened a new pull request, #13624: URL: https://github.com/apache/kafka/pull/13624 The field is not used so can be removed. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status

[jira] [Commented] (KAFKA-10883) Lien hypertext incorrect

2023-04-21 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17714998#comment-17714998 ] ASF GitHub Bot commented on KAFKA-10883: machi1990 opened a new pull request, #504: URL:

[GitHub] [kafka] urbandan commented on a diff in pull request #13475: KAFKA-14652: Add the flow to the log context and the Connect-managed …

2023-04-21 Thread via GitHub
urbandan commented on code in PR #13475: URL: https://github.com/apache/kafka/pull/13475#discussion_r1173744877 ## config/connect-mirror-maker.properties: ## @@ -57,3 +57,6 @@ config.storage.replication.factor=1 # replication.policy.separator = _ # sync.topic.acls.enabled =

[jira] [Updated] (KAFKA-14926) Remove metrics on Log Cleaner shutdown

2023-04-21 Thread Divij Vaidya (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14926?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Divij Vaidya updated KAFKA-14926: - Issue Type: Improvement (was: Bug) > Remove metrics on Log Cleaner shutdown >

[GitHub] [kafka] jlprat merged pull request #13612: MINOR: fix noticed typo in raft and metadata projects

2023-04-21 Thread via GitHub
jlprat merged PR #13612: URL: https://github.com/apache/kafka/pull/13612 -- 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:

[GitHub] [kafka] divijvaidya opened a new pull request, #13623: KAFKA-14926: Remove metrics on Log Cleaner shutdown

2023-04-21 Thread via GitHub
divijvaidya opened a new pull request, #13623: URL: https://github.com/apache/kafka/pull/13623 # Motivation When Log cleaning is shutdown, it doesn't remove metrics that were registered to `KafkaYammerMetrics.defaultRegistry()` which has one instance per server. Log cleaner's lifecycle

[GitHub] [kafka] viktorsomogyi commented on a diff in pull request #13475: KAFKA-14652: Add the flow to the log context and the Connect-managed …

2023-04-21 Thread via GitHub
viktorsomogyi commented on code in PR #13475: URL: https://github.com/apache/kafka/pull/13475#discussion_r1173739241 ## config/connect-mirror-maker.properties: ## @@ -57,3 +57,6 @@ config.storage.replication.factor=1 # replication.policy.separator = _ #

[jira] [Created] (KAFKA-14926) Remove metrics on Log Cleaner shutdown

2023-04-21 Thread Divij Vaidya (Jira)
Divij Vaidya created KAFKA-14926: Summary: Remove metrics on Log Cleaner shutdown Key: KAFKA-14926 URL: https://issues.apache.org/jira/browse/KAFKA-14926 Project: Kafka Issue Type: Bug

[jira] [Assigned] (KAFKA-12826) Remove Deprecated Class Serdes (Streams)

2023-04-21 Thread shiqin.lan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-12826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] shiqin.lan reassigned KAFKA-12826: -- Assignee: shiqin.lan > Remove Deprecated Class Serdes (Streams) >

[jira] [Assigned] (KAFKA-12826) Remove Deprecated Class Serdes (Streams)

2023-04-21 Thread shiqin.lan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-12826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] shiqin.lan reassigned KAFKA-12826: -- Assignee: (was: shiqin.lan) > Remove Deprecated Class Serdes (Streams) >

[GitHub] [kafka] dajac merged pull request #13604: KAFKA-14869: Bump coordinator value records to flexible versions (KIP…

2023-04-21 Thread via GitHub
dajac merged PR #13604: URL: https://github.com/apache/kafka/pull/13604 -- 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:

[GitHub] [kafka] dajac commented on pull request #13601: KAFKA-14869: Bump coordinator value records to flexible versions (KIP…

2023-04-21 Thread via GitHub
dajac commented on PR #13601: URL: https://github.com/apache/kafka/pull/13601#issuecomment-1517708744 @jeffkbkim Could you please rebase this one? -- 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

[GitHub] [kafka] dajac commented on pull request #13600: KAFKA-14869: Bump coordinator value records to flexible versions (KIP…

2023-04-21 Thread via GitHub
dajac commented on PR #13600: URL: https://github.com/apache/kafka/pull/13600#issuecomment-1517708617 @jeffkbkim Could you please rebase this one? -- 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

[GitHub] [kafka] dajac merged pull request #13597: KAFKA-14869: Ignore unknown record types for coordinators (KIP-915, P…

2023-04-21 Thread via GitHub
dajac merged PR #13597: URL: https://github.com/apache/kafka/pull/13597 -- 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:

[GitHub] [kafka] dajac merged pull request #13596: KAFKA-14869: Ignore unknown record types for coordinators (KIP-915, P…

2023-04-21 Thread via GitHub
dajac merged PR #13596: URL: https://github.com/apache/kafka/pull/13596 -- 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:

[GitHub] [kafka] vamossagar12 commented on pull request #13594: KAFKA-14913: Using ThreadUtils.shutdownExecutorServiceQuietly to close executors in Connect Runtime

2023-04-21 Thread via GitHub
vamossagar12 commented on PR #13594: URL: https://github.com/apache/kafka/pull/13594#issuecomment-1517696428 Thanks @yashmayya . I just tried to include only the ones which have a similar pattern. I think `SourceTaskOffsetCommitter` is something which can also be modified but I missed

[GitHub] [kafka] dajac commented on a diff in pull request #13538: KAFKA-14462; [8/N] Add ConsumerGroupMember

2023-04-21 Thread via GitHub
dajac commented on code in PR #13538: URL: https://github.com/apache/kafka/pull/13538#discussion_r1173669055 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/consumer/ConsumerGroupMember.java: ## @@ -0,0 +1,613 @@ +/* + * Licensed to the Apache Software

[GitHub] [kafka] Hangleton commented on a diff in pull request #13437: KAFKA-14828: Remove R/W locks using persistent data structures

2023-04-21 Thread via GitHub
Hangleton commented on code in PR #13437: URL: https://github.com/apache/kafka/pull/13437#discussion_r1173645096 ## metadata/src/main/java/org/apache/kafka/metadata/authorizer/AclCache.java: ## @@ -0,0 +1,107 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] Hangleton commented on pull request #13619: Initial support for OpenJDK CRaC snapshotting

2023-04-21 Thread via GitHub
Hangleton commented on PR #13619: URL: https://github.com/apache/kafka/pull/13619#issuecomment-1517679958 Hi, Radim, Thank you for the follow-up and clarifying, I missed the fact that the targeted components are the Kafka clients. Some of the previous statements regarding

[GitHub] [kafka] Hangleton commented on a diff in pull request #13437: KAFKA-14828: Remove R/W locks using persistent data structures

2023-04-21 Thread via GitHub
Hangleton commented on code in PR #13437: URL: https://github.com/apache/kafka/pull/13437#discussion_r1173645096 ## metadata/src/main/java/org/apache/kafka/metadata/authorizer/AclCache.java: ## @@ -0,0 +1,107 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[jira] [Commented] (KAFKA-14925) The website shouldn't load external resources

2023-04-21 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17714923#comment-17714923 ] Mickael Maison commented on KAFKA-14925: Done > The website shouldn't load external resources >

[jira] [Assigned] (KAFKA-14925) The website shouldn't load external resources

2023-04-21 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mickael Maison reassigned KAFKA-14925: -- Assignee: Atul Sharma > The website shouldn't load external resources >

[jira] [Commented] (KAFKA-14925) The website shouldn't load external resources

2023-04-21 Thread Atul Sharma (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17714920#comment-17714920 ] Atul Sharma commented on KAFKA-14925: - Want to work on this, can someone assign to me? > The

[GitHub] [kafka] dajac merged pull request #13537: KAFKA-14462; [7/N] Add ClientAssignor, Assignment, TopicMetadata and VersionedMetadata

2023-04-21 Thread via GitHub
dajac merged PR #13537: URL: https://github.com/apache/kafka/pull/13537 -- 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:

[GitHub] [kafka] dajac merged pull request #13555: MINOR: Move `ControllerPurgatory` to `server-common`

2023-04-21 Thread via GitHub
dajac merged PR #13555: URL: https://github.com/apache/kafka/pull/13555 -- 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:

[GitHub] [kafka] showuon commented on a diff in pull request #13514: KAFKA-14752: Kafka examples improvements - consumer changes

2023-04-21 Thread via GitHub
showuon commented on code in PR #13514: URL: https://github.com/apache/kafka/pull/13514#discussion_r1173486311 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -21,97 +21,120 @@ import org.apache.kafka.clients.consumer.ConsumerRecord; import

[GitHub] [kafka] omkreddy merged pull request #13437: KAFKA-14828: Remove R/W locks using persistent data structures

2023-04-21 Thread via GitHub
omkreddy merged PR #13437: URL: https://github.com/apache/kafka/pull/13437 -- 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:

[GitHub] [kafka] omkreddy commented on pull request #13437: KAFKA-14828: Remove R/W locks using persistent data structures

2023-04-21 Thread via GitHub
omkreddy commented on PR #13437: URL: https://github.com/apache/kafka/pull/13437#issuecomment-1517476396 Test failures are known connect/mirror failures: https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-13437/12/tests -- This is an automated message from

[GitHub] [kafka] clolov commented on pull request #13621: KAFKA-14133: Migrate ChangeLogReader mock in TaskManagerTest to Mockito

2023-04-21 Thread via GitHub
clolov commented on PR #13621: URL: https://github.com/apache/kafka/pull/13621#issuecomment-1517474714 I believe the related test failures are due to the same problem as https://github.com/apache/kafka/pull/13529#discussion_r1168432918. Once that PR is merged and this one is rebased those

[jira] [Commented] (KAFKA-14807) MirrorMaker2 config source.consumer.auto.offset.reset=latest leading to the pause of replication of consumer groups

2023-04-21 Thread Zhaoli (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17714861#comment-17714861 ] Zhaoli commented on KAFKA-14807: [~durban] Thanks, but we use dedicated mode.  > MirrorMaker2 config

[GitHub] [kafka] showuon commented on a diff in pull request #13459: KAFKA-14592: Move FeatureCommand to tools

2023-04-21 Thread via GitHub
showuon commented on code in PR #13459: URL: https://github.com/apache/kafka/pull/13459#discussion_r1173462823 ## tools/src/test/java/org/apache/kafka/tools/FeatureCommandTest.java: ## @@ -0,0 +1,292 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] [kafka] urbandan commented on a diff in pull request #13429: KAFKA-14666: Add MM2 in-memory offset translation index for offsets behind replication

2023-04-21 Thread via GitHub
urbandan commented on code in PR #13429: URL: https://github.com/apache/kafka/pull/13429#discussion_r1173450470 ## connect/mirror/src/main/java/org/apache/kafka/connect/mirror/OffsetSyncStore.java: ## @@ -139,10 +171,103 @@ public void close() { protected void

[GitHub] [kafka] urbandan commented on a diff in pull request #13429: KAFKA-14666: Add MM2 in-memory offset translation index for offsets behind replication

2023-04-21 Thread via GitHub
urbandan commented on code in PR #13429: URL: https://github.com/apache/kafka/pull/13429#discussion_r1173450470 ## connect/mirror/src/main/java/org/apache/kafka/connect/mirror/OffsetSyncStore.java: ## @@ -139,10 +171,103 @@ public void close() { protected void

[GitHub] [kafka] urbandan commented on a diff in pull request #13591: KAFKA-14831: Illegal state errors should be fatal in transactional producer

2023-04-21 Thread via GitHub
urbandan commented on code in PR #13591: URL: https://github.com/apache/kafka/pull/13591#discussion_r1173405147 ## clients/src/main/java/org/apache/kafka/clients/producer/internals/TransactionManager.java: ## @@ -968,13 +968,31 @@ private void transitionTo(State target) {

[GitHub] [kafka] emissionnebula commented on a diff in pull request #13437: KAFKA-14828: Remove R/W locks using persistent data structures

2023-04-21 Thread via GitHub
emissionnebula commented on code in PR #13437: URL: https://github.com/apache/kafka/pull/13437#discussion_r1173353923 ## server-common/src/test/java/org/apache/kafka/server/immutable/pcollections/PCollectionsImmutableMapTest.java: ## @@ -68,7 +68,7 @@ public void

  1   2   >