[GitHub] [kafka] abhijeetk88 opened a new pull request, #14127: KAFKA-15181: Wait for RemoteLogMetadataCahce to initialize after assi…

2023-07-28 Thread via GitHub
abhijeetk88 opened a new pull request, #14127: URL: https://github.com/apache/kafka/pull/14127 *More detailed description of your change, if necessary. The PR title and PR message become the squashed commit message, so use a separate comment to ping reviewers.* *Summary of

[GitHub] [kafka] satishd commented on a diff in pull request #14114: KAFKA-12969: Add broker level config synonyms for topic level tiered storage configs

2023-07-28 Thread via GitHub
satishd commented on code in PR #14114: URL: https://github.com/apache/kafka/pull/14114#discussion_r1278245511 ## core/src/test/scala/unit/kafka/log/LogConfigTest.scala: ## @@ -62,13 +63,17 @@ class LogConfigTest { kafkaProps.put(KafkaConfig.LogRollTimeJitterHoursProp,

[GitHub] [kafka] github-actions[bot] commented on pull request #12781: KAFKA-14132; Replace EasyMock with Mockito in KafkaBasedLogTest

2023-07-28 Thread via GitHub
github-actions[bot] commented on PR #12781: URL: https://github.com/apache/kafka/pull/12781#issuecomment-1656539350 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 ask a committer for review. If the PR has

[jira] [Comment Edited] (KAFKA-15252) Task is not stopped until the poll interval passes in case of task restarting.

2023-07-28 Thread Chris Egerton (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748778#comment-17748778 ] Chris Egerton edited comment on KAFKA-15252 at 7/29/23 3:12 AM: I think

[jira] [Commented] (KAFKA-15252) Task is not stopped until the poll interval passes in case of task restarting.

2023-07-28 Thread Chris Egerton (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748778#comment-17748778 ] Chris Egerton commented on KAFKA-15252: --- I think this is caused by KAFKA-15090. [~nikita.krasnov]

[GitHub] [kafka] C0urante commented on pull request #14101: Source task stop call was added to force stopping execution.

2023-07-28 Thread via GitHub
C0urante commented on PR #14101: URL: https://github.com/apache/kafka/pull/14101#issuecomment-165657 There's also [KAFKA-15090](https://issues.apache.org/jira/browse/KAFKA-15090), which gives some context on why we stopped invoking `SourceTask::stop` on the herder thread (which is the

[GitHub] [kafka] ahuang98 commented on pull request #14109: [MINOR] Addressing NPE when broker's initialCatchUpFuture fails

2023-07-28 Thread via GitHub
ahuang98 commented on PR #14109: URL: https://github.com/apache/kafka/pull/14109#issuecomment-1656471910 https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14109/2/pipeline/12 looks pretty good but I'll retrigger another build -- This is an automated

[GitHub] [kafka] junrao commented on a diff in pull request #14111: KAFKA-9800: Exponential backoff for Kafka clients - KIP-580

2023-07-28 Thread via GitHub
junrao commented on code in PR #14111: URL: https://github.com/apache/kafka/pull/14111#discussion_r1278072179 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractCoordinator.java: ## @@ -1488,9 +1500,10 @@ public void run() { }

[jira] [Assigned] (KAFKA-15271) Historicalterator can exposes elements that are too new

2023-07-28 Thread Colin McCabe (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15271?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin McCabe reassigned KAFKA-15271: Assignee: Colin McCabe > Historicalterator can exposes elements that are too new >

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1656346526 Umm build failures for jdk 8 plus some non-related test failures: ``` Build / JDK 20 and Scala 2.13 / testSyncTopicConfigs() –

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1656345210 We don't. Do you know when was is introduced? Is it in your inflight PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [kafka] jeffkbkim commented on pull request #14124: Kafka 14509

2023-07-28 Thread via GitHub
jeffkbkim commented on PR #14124: URL: https://github.com/apache/kafka/pull/14124#issuecomment-1656328786 Thanks, I will take a look next week. -- 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 #14124: Kafka 14509

2023-07-28 Thread via GitHub
dajac commented on PR #14124: URL: https://github.com/apache/kafka/pull/14124#issuecomment-1656325056 @riedelmax Thanks for the PR. I am on vacations during the next three weeks so I won’t be able to look at this one. @jeffkbkim Could you help @riedelmax? As a first step, @riedelmax

[GitHub] [kafka] mumrah opened a new pull request, #14126: MINOR Fix a Scala 2.12 checkstyle issue

2023-07-28 Thread via GitHub
mumrah opened a new pull request, #14126: URL: https://github.com/apache/kafka/pull/14126 This was introduced in #14115. I added this as part of a small tweak after the last Jenkins job. I ran `./gradlew check -x test` before committing, but apparently the issue was only present in Scala

[jira] [Updated] (KAFKA-15271) Historicalterator can exposes elements that are too new

2023-07-28 Thread Colin McCabe (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15271?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin McCabe updated KAFKA-15271: - Summary: Historicalterator can exposes elements that are too new (was:

[jira] [Comment Edited] (KAFKA-14509) Add ConsumerGroupDescribe API

2023-07-28 Thread Max Riedel (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748724#comment-17748724 ] Max Riedel edited comment on KAFKA-14509 at 7/28/23 7:51 PM: - [~dajac]

[jira] [Comment Edited] (KAFKA-14509) Add ConsumerGroupDescribe API

2023-07-28 Thread Max Riedel (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748724#comment-17748724 ] Max Riedel edited comment on KAFKA-14509 at 7/28/23 7:50 PM: - [~dajac]

[jira] [Commented] (KAFKA-14509) Add ConsumerGroupDescribe API

2023-07-28 Thread Max Riedel (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748724#comment-17748724 ] Max Riedel commented on KAFKA-14509: [~dajac] Thanks for the helpful comments. Please see my PR for

[GitHub] [kafka] riedelmax opened a new pull request, #14124: Kafka 14509

2023-07-28 Thread via GitHub
riedelmax opened a new pull request, #14124: URL: https://github.com/apache/kafka/pull/14124 *More detailed description of your change, if necessary. The PR title and PR message become the squashed commit message, so use a separate comment to ping reviewers.* *Summary of

[GitHub] [kafka] gharris1727 commented on pull request #14101: Source task stop call was added to force stopping execution.

2023-07-28 Thread via GitHub
gharris1727 commented on PR #14101: URL: https://github.com/apache/kafka/pull/14101#issuecomment-1656189960 Hey thanks @akitoshka for giving some attention to this issue. Unfortunately the fix as-is can't be merged, for the reasons that @yashmayya and @vamossagar12 raised above.

[jira] [Resolved] (KAFKA-15263) KRaftMigrationDriver can run the migration twice

2023-07-28 Thread David Arthur (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Arthur resolved KAFKA-15263. -- Resolution: Fixed > KRaftMigrationDriver can run the migration twice >

[GitHub] [kafka] gharris1727 merged pull request #14091: MINOR: Remove duplicate instantiation of MockConnectMetrics in AbstractWorkerSourceTaskTest

2023-07-28 Thread via GitHub
gharris1727 merged PR #14091: URL: https://github.com/apache/kafka/pull/14091 -- 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] gharris1727 commented on pull request #14091: MINOR: Remove duplicate instantiation of MockConnectMetrics in AbstractWorkerSourceTaskTest

2023-07-28 Thread via GitHub
gharris1727 commented on PR #14091: URL: https://github.com/apache/kafka/pull/14091#issuecomment-1656157411 Flaky test failures appear unrelated, and the JDK8 build succeeded with no failures! -- This is an automated message from the Apache Git Service. To respond to the message, please

[jira] [Updated] (KAFKA-15263) KRaftMigrationDriver can run the migration twice

2023-07-28 Thread David Arthur (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Arthur updated KAFKA-15263: - Affects Version/s: 3.5.1 3.5.0 > KRaftMigrationDriver can run the

[GitHub] [kafka] jolshan merged pull request #14117: MINOR: Code cleanups in group-coordinator module

2023-07-28 Thread via GitHub
jolshan merged PR #14117: URL: https://github.com/apache/kafka/pull/14117 -- 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] [Updated] (KAFKA-15263) KRaftMigrationDriver can run the migration twice

2023-07-28 Thread David Arthur (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Arthur updated KAFKA-15263: - Fix Version/s: 3.6.0 3.5.2 > KRaftMigrationDriver can run the migration

[jira] [Resolved] (KAFKA-14702) Extend server side assignor to support rack aware replica placement

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14702?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot resolved KAFKA-14702. - Fix Version/s: 3.6.0 Resolution: Fixed > Extend server side assignor to support rack

[GitHub] [kafka] dajac commented on pull request #14099: KAFKA-14702: Extend server side assignor to support rack aware replica placement

2023-07-28 Thread via GitHub
dajac commented on PR #14099: URL: https://github.com/apache/kafka/pull/14099#issuecomment-1656057225 The build for JDK 17 is stuck... The three others look good. I have seen a few build stuck this week in other PRs so this is clearly not related to changes made in this one. Moreover, we

[GitHub] [kafka] philipnee commented on a diff in pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee commented on code in PR #14123: URL: https://github.com/apache/kafka/pull/14123#discussion_r1277840446 ## clients/src/test/java/org/apache/kafka/clients/consumer/internals/PrototypeAsyncConsumerTest.java: ## @@ -153,12 +153,14 @@ public void

[GitHub] [kafka] philipnee commented on a diff in pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee commented on code in PR #14123: URL: https://github.com/apache/kafka/pull/14123#discussion_r1277835494 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/events/OffsetFetchApplicationEvent.java: ## @@ -19,25 +19,33 @@ import

[GitHub] [kafka] philipnee opened a new pull request, #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee opened a new pull request, #14123: URL: https://github.com/apache/kafka/pull/14123 **Summary** I discovered the committed() API timeout during the integration test. After investigation, this is because the future was not completed in the ApplicationEventProcessor. I also

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Add comment to onPartitionsLost override

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277811461 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] dajac commented on a diff in pull request #14121: MINOR: Add comment to onPartitionsLost override

2023-07-28 Thread via GitHub
dajac commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277803779 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Add comment to onPartitionsLost override

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277800358 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] mumrah merged pull request #14115: KAFKA-15263 Check KRaftMigrationDriver state in each event

2023-07-28 Thread via GitHub
mumrah merged PR #14115: URL: https://github.com/apache/kafka/pull/14115 -- 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] mumrah commented on pull request #14115: KAFKA-15263 Check KRaftMigrationDriver state in each event

2023-07-28 Thread via GitHub
mumrah commented on PR #14115: URL: https://github.com/apache/kafka/pull/14115#issuecomment-1656019490 Test failures look unrelated https://github.com/apache/kafka/assets/55116/a8e1f6db-57ef-4450-b595-a54c164db524;> -- This is an automated message from the Apache Git Service.

[GitHub] [kafka] dajac commented on a diff in pull request #14121: MINOR: Add comment to onPartitionsLost override

2023-07-28 Thread via GitHub
dajac commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277792665 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] dajac commented on a diff in pull request #14121: MINOR: Add comment to onPartitionsLost override

2023-07-28 Thread via GitHub
dajac commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277792665 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Add comment to onPartitionsLost override

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[jira] [Commented] (KAFKA-14257) Unexpected error INCONSISTENT_CLUSTER_ID in VOTE response

2023-07-28 Thread jianbin.chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748684#comment-17748684 ] jianbin.chen commented on KAFKA-14257: -- [~cokutan] You'll want to use the same meta.properties,

[jira] [Updated] (KAFKA-15264) Compared with 1.1.0zk, the peak throughput of 3.5.1kraft is very jitter

2023-07-28 Thread jianbin.chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] jianbin.chen updated KAFKA-15264: - Description: I was preparing to upgrade from 1.1.0 to 3.5.1 kraft mode (new cluster

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] jeffkbkim opened a new pull request, #14122: MINOR: GroupMetadataManagerTest.java style fix

2023-07-28 Thread via GitHub
jeffkbkim opened a new pull request, #14122: URL: https://github.com/apache/kafka/pull/14122 This patch makes the styling consistent inside GroupMetadataManagerTest.java. Also, it adds `JoinResult` to simplify the JoinGroup API responses in the tests. ### Committer Checklist

[GitHub] [kafka] fvaleri commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277749103 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[GitHub] [kafka] junrao merged pull request #14086: MINOR: Test assign() and assignment() in the integration test

2023-07-28 Thread via GitHub
junrao merged PR #14086: URL: https://github.com/apache/kafka/pull/14086 -- 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] jolshan commented on a diff in pull request #14117: MINOR: Code cleanups in group-coordinator module

2023-07-28 Thread via GitHub
jolshan commented on code in PR #14117: URL: https://github.com/apache/kafka/pull/14117#discussion_r1277712647 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/runtime/CoordinatorRuntime.java: ## @@ -1014,7 +1013,7 @@ public void onHighWatermarkUpdated(

[GitHub] [kafka] dajac commented on a diff in pull request #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
dajac commented on code in PR #14121: URL: https://github.com/apache/kafka/pull/14121#discussion_r1277683057 ## examples/src/main/java/kafka/examples/Consumer.java: ## @@ -157,9 +157,4 @@ public void onPartitionsRevoked(Collection partitions) { public void

[jira] [Updated] (KAFKA-15271) TimelineHashMap.entrySet yield unexpected results with nested TimelineHashMap

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15271?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-15271: Description: Example: {code:java} @Test public void bug() { SnapshotRegistry

[GitHub] [kafka] clolov commented on pull request #13260: KAFKA-14661: Upgrade Zookeeper to 3.8.1

2023-07-28 Thread via GitHub
clolov commented on PR #13260: URL: https://github.com/apache/kafka/pull/13260#issuecomment-1655850239 Zoning in on ControllerIntegrationTest#testControllerMoveOnTopicCreation: ``` ... val event = appender.getMessages.find(e => e.getLevel == Level.INFO &&

[jira] [Created] (KAFKA-15271) TimelineHashMap.entrySet yield unexpected results with nested TimelineHashMap

2023-07-28 Thread David Jacot (Jira)
David Jacot created KAFKA-15271: --- Summary: TimelineHashMap.entrySet yield unexpected results with nested TimelineHashMap Key: KAFKA-15271 URL: https://issues.apache.org/jira/browse/KAFKA-15271 Project:

[jira] [Assigned] (KAFKA-15270) Integration tests for AsyncConsumer simple consume case

2023-07-28 Thread Lianet Magrans (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15270?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lianet Magrans reassigned KAFKA-15270: -- Assignee: Lianet Magrans > Integration tests for AsyncConsumer simple consume case >

[jira] [Created] (KAFKA-15270) Integration tests for AsyncConsumer simple consume case

2023-07-28 Thread Lianet Magrans (Jira)
Lianet Magrans created KAFKA-15270: -- Summary: Integration tests for AsyncConsumer simple consume case Key: KAFKA-15270 URL: https://issues.apache.org/jira/browse/KAFKA-15270 Project: Kafka

[GitHub] [kafka] clolov commented on pull request #13260: KAFKA-14661: Upgrade Zookeeper to 3.8.1

2023-07-28 Thread via GitHub
clolov commented on PR #13260: URL: https://github.com/apache/kafka/pull/13260#issuecomment-1655821383 At least on a first look it appears something changed/is wrong with the loggers: ``` [2023-02-22T17:35:34.655Z] Gradle Test Run :core:unitTest > Gradle Test Executor 136 >

[jira] [Created] (KAFKA-15269) Clean up the RaftClient interface

2023-07-28 Thread Jira
José Armando García Sancio created KAFKA-15269: -- Summary: Clean up the RaftClient interface Key: KAFKA-15269 URL: https://issues.apache.org/jira/browse/KAFKA-15269 Project: Kafka

[GitHub] [kafka] clolov commented on pull request #13260: KAFKA-14661: Upgrade Zookeeper to 3.8.1

2023-07-28 Thread via GitHub
clolov commented on PR #13260: URL: https://github.com/apache/kafka/pull/13260#issuecomment-1655753598 At least the below tests have started to continuously fail. I am still looking as to why: ``` Gradle Test Run :core:test > Gradle Test Executor 3 > ControllerIntegrationTest >

[jira] [Assigned] (KAFKA-15089) Consolidate all the group coordinator configs

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15089?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot reassigned KAFKA-15089: --- Assignee: (was: David Jacot) > Consolidate all the group coordinator configs >

[jira] [Assigned] (KAFKA-15225) Define constants for record types

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15225?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot reassigned KAFKA-15225: --- Assignee: (was: David Jacot) > Define constants for record types >

[jira] [Assigned] (KAFKA-15061) CoordinatorPartitionWriter should reuse buffer

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15061?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot reassigned KAFKA-15061: --- Assignee: (was: David Jacot) > CoordinatorPartitionWriter should reuse buffer >

[jira] [Updated] (KAFKA-15225) Define constants for record types

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15225?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-15225: Priority: Minor (was: Major) > Define constants for record types >

[jira] [Created] (KAFKA-15268) Consider replacing Subscription Metadata by a hash

2023-07-28 Thread David Jacot (Jira)
David Jacot created KAFKA-15268: --- Summary: Consider replacing Subscription Metadata by a hash Key: KAFKA-15268 URL: https://issues.apache.org/jira/browse/KAFKA-15268 Project: Kafka Issue Type:

[jira] [Commented] (KAFKA-14509) Add ConsumerGroupDescribe API

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748614#comment-17748614 ] David Jacot commented on KAFKA-14509: - [~riedelmax] I left a few comments in your comment. > Add

[jira] [Commented] (KAFKA-14509) Add ConsumerGroupDescribe API

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748613#comment-17748613 ] David Jacot commented on KAFKA-14509: - [~riedelmax] There are no error codes in requests so I am not

[jira] [Resolved] (KAFKA-14501) Implement Heartbeat API

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14501?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot resolved KAFKA-14501. - Fix Version/s: 3.6.0 Reviewer: David Jacot Resolution: Fixed > Implement

[GitHub] [kafka] dajac merged pull request #14056: KAFKA-14501: Implement Heartbeat protocol in new GroupCoordinator

2023-07-28 Thread via GitHub
dajac merged PR #14056: URL: https://github.com/apache/kafka/pull/14056 -- 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] fvaleri opened a new pull request, #14121: MINOR: Remove onPartitionsLost overriding in favor of default implementation

2023-07-28 Thread via GitHub
fvaleri opened a new pull request, #14121: URL: https://github.com/apache/kafka/pull/14121 (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,

[GitHub] [kafka] dajac commented on a diff in pull request #14056: KAFKA-14501: Implement Heartbeat protocol in new GroupCoordinator

2023-07-28 Thread via GitHub
dajac commented on code in PR #14056: URL: https://github.com/apache/kafka/pull/14056#discussion_r1277524963 ## group-coordinator/src/test/java/org/apache/kafka/coordinator/group/GroupMetadataManagerTest.java: ## @@ -843,15 +886,19 @@ public JoinGroupResponseData

[jira] [Updated] (KAFKA-15267) Cluster-wide disablement of Tiered Storage

2023-07-28 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15267?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-15267: -- Labels: tiered-storage (was: ) > Cluster-wide disablement of Tiered Storage >

[jira] [Updated] (KAFKA-15267) Cluster-wide disablement of Tiered Storage

2023-07-28 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15267?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-15267: -- Description: h2. Summary KIP-405 defines the configuration

[jira] [Created] (KAFKA-15267) Cluster-wide disablement of Tiered Storage

2023-07-28 Thread Christo Lolov (Jira)
Christo Lolov created KAFKA-15267: - Summary: Cluster-wide disablement of Tiered Storage Key: KAFKA-15267 URL: https://issues.apache.org/jira/browse/KAFKA-15267 Project: Kafka Issue Type:

[GitHub] [kafka] dajac commented on a diff in pull request #14117: MINOR: Code cleanups in group-coordinator module

2023-07-28 Thread via GitHub
dajac commented on code in PR #14117: URL: https://github.com/apache/kafka/pull/14117#discussion_r1277523528 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/runtime/CoordinatorBuilderSupplier.java: ## @@ -17,14 +17,14 @@ package

[GitHub] [kafka] jeffkbkim commented on a diff in pull request #14117: MINOR: Code cleanups in group-coordinator module

2023-07-28 Thread via GitHub
jeffkbkim commented on code in PR #14117: URL: https://github.com/apache/kafka/pull/14117#discussion_r1277519026 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/runtime/CoordinatorBuilderSupplier.java: ## @@ -17,14 +17,14 @@ package

[jira] [Resolved] (KAFKA-15246) CoordinatorContext should be protected by a lock

2023-07-28 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15246?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot resolved KAFKA-15246. - Fix Version/s: 3.6.0 Reviewer: Justine Olshan Resolution: Fixed >

[GitHub] [kafka] dajac opened a new pull request, #14120: KAFKA-14499: [4/N] Implement OffsetFetch API

2023-07-28 Thread via GitHub
dajac opened a new pull request, #14120: URL: https://github.com/apache/kafka/pull/14120 This patch implements the OffsetFetch API in the new group coordinator. I found out that implementing the `RequireStable` flag is hard (to not say impossible) in the current model. For the

[GitHub] [kafka] amangandhi94 opened a new pull request, #14119: KAFKA-15266: Static configs set for non primary synonyms are ignored for Log configs

2023-07-28 Thread via GitHub
amangandhi94 opened a new pull request, #14119: URL: https://github.com/apache/kafka/pull/14119 This change ensures that values stored in "non-primary" synonyms is respected when altering configs. Currently if "non-primary" synonyms have a static config set up then it gets deleted from the

[jira] [Commented] (KAFKA-15259) Kafka Streams does not continue processing due to rollback despite ProductionExceptionHandlerResponse.CONTINUE if using execute_once

2023-07-28 Thread Tomonari Yamashita (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17748555#comment-17748555 ] Tomonari Yamashita commented on KAFKA-15259: Hi [~mjsax], I think the stacktrace can be

[jira] [Updated] (KAFKA-15266) Log configs ignore static configs set for non primary synonyms

2023-07-28 Thread Aman Harish Gandhi (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-15266?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aman Harish Gandhi updated KAFKA-15266: --- Summary: Log configs ignore static configs set for non primary synonyms (was: Log

[jira] [Created] (KAFKA-15266) Log configs ignore static configs set non primary synonyms

2023-07-28 Thread Aman Harish Gandhi (Jira)
Aman Harish Gandhi created KAFKA-15266: -- Summary: Log configs ignore static configs set non primary synonyms Key: KAFKA-15266 URL: https://issues.apache.org/jira/browse/KAFKA-15266 Project: Kafka

[jira] [Assigned] (KAFKA-14967) MockAdminClient throws NullPointerException in CreateTopicsResult

2023-07-28 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14967?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mickael Maison reassigned KAFKA-14967: -- Fix Version/s: 3.6.0 Assignee: James Shaw Resolution: Fixed >

[GitHub] [kafka] mimaison merged pull request #13671: KAFKA-14967: fix NPE in MockAdminClient CreateTopicsResult

2023-07-28 Thread via GitHub
mimaison merged PR #13671: URL: https://github.com/apache/kafka/pull/13671 -- 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 #14101: Source task stop call was added to force stopping execution.

2023-07-28 Thread via GitHub
vamossagar12 commented on PR #14101: URL: https://github.com/apache/kafka/pull/14101#issuecomment-1655210284 > This is exactly our case: we have long source polling (in our case it's jdbc source task), each restarting doesn't break previous task (we get Graceful Shutdown failed), but

[GitHub] [kafka] dajac commented on pull request #14117: MINOR: Code cleanups in group-coordinator module

2023-07-28 Thread via GitHub
dajac commented on PR #14117: URL: https://github.com/apache/kafka/pull/14117#issuecomment-1655195050 > can you point me to where the log context includes the topic partition info? Sure. It is right

[GitHub] [kafka] flashmouse commented on pull request #13920: KAFKA-15106 fix AbstractStickyAssignor isBalanced predict

2023-07-28 Thread via GitHub
flashmouse commented on PR #13920: URL: https://github.com/apache/kafka/pull/13920#issuecomment-1655170419 thx @ableegoldman I reduce partitions num and increase consumers num, I think all tests influenced by this pr should be PASSED now. -- This is an automated message from the

[GitHub] [kafka] muralibasani commented on pull request #14106: KAFKA-14585: 1st part : Java versions for metadata/broker and updated LogConfig

2023-07-28 Thread via GitHub
muralibasani commented on PR #14106: URL: https://github.com/apache/kafka/pull/14106#issuecomment-1655168025 @fvaleri would like to take a look at 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