[jira] [Updated] (KAFKA-14691) OffsetFetch API Version 9

2023-02-07 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14691?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-14691: Description: The goal of this jira is to implement the version 9 of the OffsetFetch API as

[jira] [Updated] (KAFKA-14690) OffsetCommit API Version 9

2023-02-07 Thread David Jacot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14690?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Jacot updated KAFKA-14690: Description: The goal of this jira is to implement the version 9 of the OffsetCommit API as

[jira] [Created] (KAFKA-14691) OffsetFetch API Version 9

2023-02-07 Thread David Jacot (Jira)
David Jacot created KAFKA-14691: --- Summary: OffsetFetch API Version 9 Key: KAFKA-14691 URL: https://issues.apache.org/jira/browse/KAFKA-14691 Project: Kafka Issue Type: Sub-task

[jira] [Created] (KAFKA-14690) OffsetCommit API Version 9

2023-02-07 Thread David Jacot (Jira)
David Jacot created KAFKA-14690: --- Summary: OffsetCommit API Version 9 Key: KAFKA-14690 URL: https://issues.apache.org/jira/browse/KAFKA-14690 Project: Kafka Issue Type: Sub-task

[GitHub] [kafka] fvaleri commented on pull request #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`

2023-02-07 Thread via GitHub
fvaleri commented on PR #13216: URL: https://github.com/apache/kafka/pull/13216#issuecomment-1422141061 Hi @scholzj, thanks. I think this is not needed anymore, but we should probably keep the system tests aligned: `tests/kafkatest/services/templates/connect_log4j.properties`. -- This

[jira] [Updated] (KAFKA-14401) Connector/Tasks reading offsets can get stuck if underneath WorkThread dies

2023-02-07 Thread Sagar Rao (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14401?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sagar Rao updated KAFKA-14401: -- Labels: (was: KafkaConnect) > Connector/Tasks reading offsets can get stuck if underneath

[jira] [Updated] (KAFKA-14401) Connector/Tasks reading offsets can get stuck if underneath WorkThread dies

2023-02-07 Thread Sagar Rao (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14401?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sagar Rao updated KAFKA-14401: -- Component/s: KafkaConnect > Connector/Tasks reading offsets can get stuck if underneath WorkThread

[jira] [Updated] (KAFKA-14401) Connector/Tasks reading offsets can get stuck if underneath WorkThread dies

2023-02-07 Thread Sagar Rao (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14401?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sagar Rao updated KAFKA-14401: -- Labels: KafkaConnect (was: Connect) > Connector/Tasks reading offsets can get stuck if underneath

[jira] [Updated] (KAFKA-14401) Connector/Tasks reading offsets can get stuck if underneath WorkThread dies

2023-02-07 Thread Sagar Rao (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14401?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sagar Rao updated KAFKA-14401: -- Labels: Connect (was: ) > Connector/Tasks reading offsets can get stuck if underneath WorkThread

[jira] [Updated] (KAFKA-14660) Divide by zero security vulnerability (sonatype-2019-0422)

2023-02-07 Thread Matthias J. Sax (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias J. Sax updated KAFKA-14660: Fix Version/s: 3.4.1 > Divide by zero security vulnerability (sonatype-2019-0422) >

[GitHub] [kafka] mjsax commented on pull request #13175: KAFAK-14660: Fix divide-by-zero vulnerability

2023-02-07 Thread via GitHub
mjsax commented on PR #13175: URL: https://github.com/apache/kafka/pull/13175#issuecomment-1422045815 Cherry-picked to `3.4` branch. -- 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

[GitHub] [kafka] mattwong949 commented on a diff in pull request #13206: [KAFKA-14685] Refactor logic to handle OFFSET_MOVED_TO_TIERED_STORAGE error

2023-02-07 Thread via GitHub
mattwong949 commented on code in PR #13206: URL: https://github.com/apache/kafka/pull/13206#discussion_r1099642781 ## core/src/main/java/kafka/server/ReplicaFetcherTierStateMachine.java: ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [kafka] mattwong949 commented on a diff in pull request #13206: [KAFKA-14685] Refactor logic to handle OFFSET_MOVED_TO_TIERED_STORAGE error

2023-02-07 Thread via GitHub
mattwong949 commented on code in PR #13206: URL: https://github.com/apache/kafka/pull/13206#discussion_r1099642781 ## core/src/main/java/kafka/server/ReplicaFetcherTierStateMachine.java: ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [kafka] mattwong949 commented on a diff in pull request #13206: [KAFKA-14685] Refactor logic to handle OFFSET_MOVED_TO_TIERED_STORAGE error

2023-02-07 Thread via GitHub
mattwong949 commented on code in PR #13206: URL: https://github.com/apache/kafka/pull/13206#discussion_r1099641934 ## core/src/main/java/kafka/server/ReplicaFetcherTierStateMachine.java: ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [kafka] mattwong949 commented on a diff in pull request #13206: [KAFKA-14685] Refactor logic to handle OFFSET_MOVED_TO_TIERED_STORAGE error

2023-02-07 Thread via GitHub
mattwong949 commented on code in PR #13206: URL: https://github.com/apache/kafka/pull/13206#discussion_r1099635815 ## core/src/main/scala/kafka/server/AbstractFetcherThread.scala: ## @@ -400,12 +386,7 @@ abstract class AbstractFetcherThread(name: String, case

[GitHub] [kafka] satishd commented on a diff in pull request #13046: KAFKA-14551 Move/Rewrite LeaderEpochFileCache and its dependencies to the storage module.

2023-02-07 Thread via GitHub
satishd commented on code in PR #13046: URL: https://github.com/apache/kafka/pull/13046#discussion_r1099624434 ## storage/src/main/java/org/apache/kafka/server/log/internals/LeaderEpochCheckpoint.java: ## @@ -0,0 +1,26 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [kafka] satishd merged pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd merged PR #13213: URL: https://github.com/apache/kafka/pull/13213 -- 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] satishd commented on pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd commented on PR #13213: URL: https://github.com/apache/kafka/pull/13213#issuecomment-1421954683 Test failure is not related to this change, will merge the changes to trunk. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [kafka] ijuma opened a new pull request, #13218: MINOR: Remove unnecessary usage of `LazyIndex`

2023-02-07 Thread via GitHub
ijuma opened a new pull request, #13218: URL: https://github.com/apache/kafka/pull/13218 The remote index classes use `LazyIndex`, but immediately force materialization. This results in more verbose code and it's misleading since the indexes are not lazily used in practice. Also

[jira] [Resolved] (KAFKA-14689) Kafka won't start on Windows in KRaft mode

2023-02-07 Thread Deng Ziming (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Deng Ziming resolved KAFKA-14689. - Resolution: Duplicate This is the same with KAFKA-14273 > Kafka won't start on Windows in

[jira] [Resolved] (KAFKA-14687) Possible issue on documentation : producerConfig -> retries

2023-02-07 Thread Ahmed Toumi (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14687?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ahmed Toumi resolved KAFKA-14687. - Resolution: Fixed I see that was fixed at this revision number :

[jira] [Comment Edited] (KAFKA-14687) Possible issue on documentation : producerConfig -> retries

2023-02-07 Thread Ahmed Toumi (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14687?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17685610#comment-17685610 ] Ahmed Toumi edited comment on KAFKA-14687 at 2/7/23 11:33 PM: -- I see that

[GitHub] [kafka] vcrfxia commented on a diff in pull request #13188: KAFKA-14491: [5/N] Basic operations for RocksDB versioned store

2023-02-07 Thread via GitHub
vcrfxia commented on code in PR #13188: URL: https://github.com/apache/kafka/pull/13188#discussion_r1099321787 ## streams/src/main/java/org/apache/kafka/streams/state/internals/RocksDBVersionedStore.java: ## @@ -0,0 +1,792 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] [kafka] philipnee opened a new pull request, #13217: Prototype take 2

2023-02-07 Thread via GitHub
philipnee opened a new pull request, #13217: URL: https://github.com/apache/kafka/pull/13217 *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] guozhangwang commented on pull request #13025: KAFKA-14299: Fix pause and resume with state updater

2023-02-07 Thread via GitHub
guozhangwang commented on PR #13025: URL: https://github.com/apache/kafka/pull/13025#issuecomment-1421340480 I do not have further comments. @cadonna do you want to take a final look? If not I can go ahead and merge it. -- This is an automated message from the Apache Git Service. To

[GitHub] [kafka] philipnee commented on pull request #13021: KAFKA-14468: Implement CommitRequestManager to manage the commit and autocommit requests

2023-02-07 Thread via GitHub
philipnee commented on PR #13021: URL: https://github.com/apache/kafka/pull/13021#issuecomment-1421340357 much thanks @guozhangwang -- 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

[GitHub] [kafka] scholzj opened a new pull request, #13216: Remove unused ZooKeeper log level configuration from `connect-log4j.properties`

2023-02-07 Thread via GitHub
scholzj opened a new pull request, #13216: URL: https://github.com/apache/kafka/pull/13216 The `connect-log4j.properties` file configures the log level for the ZooKeeper logger. That seems unnecessary since Kafka Connect does not use ZooKeeper directly in any way. This PR removes it from

[GitHub] [kafka] guozhangwang commented on pull request #13179: KAFKA-10575: Add onRestoreSuspsnded to StateRestoreListener

2023-02-07 Thread via GitHub
guozhangwang commented on PR #13179: URL: https://github.com/apache/kafka/pull/13179#issuecomment-1421336158 Merged to trunk after updating the javadoc. -- 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

[GitHub] [kafka] guozhangwang merged pull request #13179: KAFKA-10575: Add onRestoreSuspsnded to StateRestoreListener

2023-02-07 Thread via GitHub
guozhangwang merged PR #13179: URL: https://github.com/apache/kafka/pull/13179 -- 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] guozhangwang commented on pull request #13021: KAFKA-14468: Implement CommitRequestManager to manage the commit and autocommit requests

2023-02-07 Thread via GitHub
guozhangwang commented on PR #13021: URL: https://github.com/apache/kafka/pull/13021#issuecomment-1421334205 Made another look, LGTM. Leaving for @hachikuji for a final look. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [kafka] guozhangwang commented on a diff in pull request #13021: KAFKA-14468: Implement CommitRequestManager to manage the commit and autocommit requests

2023-02-07 Thread via GitHub
guozhangwang commented on code in PR #13021: URL: https://github.com/apache/kafka/pull/13021#discussion_r1096350392 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/CommitRequestManager.java: ## @@ -0,0 +1,224 @@ +/* + * Licensed to the Apache Software

[GitHub] [kafka] mjsax merged pull request #13209: MINOR: remove unnecessary helper method

2023-02-07 Thread via GitHub
mjsax merged PR #13209: URL: https://github.com/apache/kafka/pull/13209 -- 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 #12813: KAFKA-14317: ProduceRequest timeouts are logged as network exceptions

2023-02-07 Thread via GitHub
dajac commented on PR #12813: URL: https://github.com/apache/kafka/pull/12813#issuecomment-1421316241 @guozhangwang Thanks. I will do this tomorrow. In the mean time, I have re-triggered the CI as they were multiple failures. -- This is an automated message from the Apache Git Service.

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1099087860 ## core/src/test/scala/unit/kafka/server/ApiVersionsRequestTest.scala: ## @@ -44,6 +44,13 @@ class ApiVersionsRequestTest(cluster: ClusterInstance) extends

[GitHub] [kafka] guozhangwang commented on pull request #12813: KAFKA-14317: ProduceRequest timeouts are logged as network exceptions

2023-02-07 Thread via GitHub
guozhangwang commented on PR #12813: URL: https://github.com/apache/kafka/pull/12813#issuecomment-1421307753 I made another pass and it LGTM now. leaving to @dajac for a final look and merge. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [kafka] guozhangwang commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
guozhangwang commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1099085546 ## core/src/test/scala/unit/kafka/server/ApiVersionsRequestTest.scala: ## @@ -44,6 +44,13 @@ class ApiVersionsRequestTest(cluster: ClusterInstance) extends

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1099084820 ## generator/src/main/java/org/apache/kafka/message/ApiMessageTypeGenerator.java: ## @@ -408,6 +410,23 @@ private void generateListenerTypesEnum() {

[jira] [Assigned] (KAFKA-14648) Do not fail clients if bootstrap servers is not immediately resolvable

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14648?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk True reassigned KAFKA-14648: - Assignee: Philip Nee (was: Kirk True) > Do not fail clients if bootstrap servers is not

[GitHub] [kafka] guozhangwang commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
guozhangwang commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1099081262 ## clients/src/main/resources/common/message/ConsumerGroupHeartbeatRequest.json: ## @@ -0,0 +1,66 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] [kafka] hachikuji commented on a diff in pull request #13207: KAFKA-14664; Fix inaccurate raft idle ratio metric

2023-02-07 Thread via GitHub
hachikuji commented on code in PR #13207: URL: https://github.com/apache/kafka/pull/13207#discussion_r1099065953 ## raft/src/main/java/org/apache/kafka/raft/internals/KafkaRaftMetrics.java: ## @@ -133,26 +131,27 @@ public KafkaRaftMetrics(Metrics metrics, String

[jira] [Assigned] (KAFKA-14648) Do not fail clients if bootstrap servers is not immediately resolvable

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14648?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk True reassigned KAFKA-14648: - Assignee: Kirk True > Do not fail clients if bootstrap servers is not immediately resolvable >

[GitHub] [kafka] hachikuji commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
hachikuji commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1099052637 ## generator/src/main/java/org/apache/kafka/message/ApiMessageTypeGenerator.java: ## @@ -408,6 +410,23 @@ private void generateListenerTypesEnum() {

[jira] [Comment Edited] (KAFKA-14648) Do not fail clients if bootstrap servers is not immediately resolvable

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17685418#comment-17685418 ] Kirk True edited comment on KAFKA-14648 at 2/7/23 6:30 PM: --- Would a fix for

[jira] [Comment Edited] (KAFKA-14648) Do not fail clients if bootstrap servers is not immediately resolvable

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17685418#comment-17685418 ] Kirk True edited comment on KAFKA-14648 at 2/7/23 6:29 PM: --- Would a fix for

[jira] [Commented] (KAFKA-14648) Do not fail clients if bootstrap servers is not immediately resolvable

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17685418#comment-17685418 ] Kirk True commented on KAFKA-14648: --- Would a fix for this cover the case where

[GitHub] [kafka] kirktrue commented on pull request #13211: KAFKA-14676: Include all SASL configs in login cache key to ensure clients in a JVM can use different OAuth configs

2023-02-07 Thread via GitHub
kirktrue commented on PR #13211: URL: https://github.com/apache/kafka/pull/13211#issuecomment-1421255523 @rajinisivaram - LGTM. It took me a second to verify that the `configs` that was passed into the `LoginMetadata ` wasn't being used directly in the `hashCode` and `equals`

[GitHub] [kafka] jsancio commented on a diff in pull request #13207: KAFKA-14664; Fix inaccurate raft idle ratio metric

2023-02-07 Thread via GitHub
jsancio commented on code in PR #13207: URL: https://github.com/apache/kafka/pull/13207#discussion_r1098994575 ## raft/src/main/java/org/apache/kafka/raft/internals/KafkaRaftMetrics.java: ## @@ -133,26 +131,27 @@ public KafkaRaftMetrics(Metrics metrics, String metricGrpPrefix,

[GitHub] [kafka] fvaleri commented on pull request #13215: KAFKA-14578: Move ConsumerPerformance to tools

2023-02-07 Thread via GitHub
fvaleri commented on PR #13215: URL: https://github.com/apache/kafka/pull/13215#issuecomment-1421199547 @mimaison @OmniaGM @clolov -- 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

[GitHub] [kafka] fvaleri opened a new pull request, #13215: KAFKA-14578: Move ConsumerPerformance to tools

2023-02-07 Thread via GitHub
fvaleri opened a new pull request, #13215: URL: https://github.com/apache/kafka/pull/13215 ```sh === test data bin/kafka-producer-perf-test.sh --num-records 3 --record-size 1000 --throughput 200 --topic my-topic --producer-props bootstrap.servers=:9092 linger.ms=0

[GitHub] [kafka] guozhangwang commented on pull request #13125: KAFKA-14626: Kafka Consumer Coordinator does not cleanup all metrics after shutdown

2023-02-07 Thread via GitHub
guozhangwang commented on PR #13125: URL: https://github.com/apache/kafka/pull/13125#issuecomment-1421133959 @yufeiyan1220 just following up on this PR. Do you want to address the latest comment, I'm thinking if we can just move `throttleTimeSensor` to the Selector while keeping its group

[GitHub] [kafka] dajac commented on pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on PR #12972: URL: https://github.com/apache/kafka/pull/12972#issuecomment-1421105810 @hachikuji Thanks for your review. I have addressed your comments. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [kafka] mimaison merged pull request #13210: MINOR: Remove accidental unnecessary code

2023-02-07 Thread via GitHub
mimaison merged PR #13210: URL: https://github.com/apache/kafka/pull/13210 -- 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-14689) Kafka won't start on Windows in KRaft mode

2023-02-07 Thread Kedar Joshi (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kedar Joshi updated KAFKA-14689: Description: Kafka 3.4 is unable to start in KRaft mode on Windows 10. Kafka is started in Kafka

[jira] [Created] (KAFKA-14689) Kafka won't start on Windows in KRaft mode

2023-02-07 Thread Kedar Joshi (Jira)
Kedar Joshi created KAFKA-14689: --- Summary: Kafka won't start on Windows in KRaft mode Key: KAFKA-14689 URL: https://issues.apache.org/jira/browse/KAFKA-14689 Project: Kafka Issue Type: Bug

[jira] [Assigned] (KAFKA-14484) Move UnifiedLog to storage module

2023-02-07 Thread Alexandre Dupriez (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alexandre Dupriez reassigned KAFKA-14484: - Assignee: Alexandre Dupriez > Move UnifiedLog to storage module >

[jira] [Assigned] (KAFKA-14482) Move LogLoader to storage module

2023-02-07 Thread Alexandre Dupriez (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14482?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alexandre Dupriez reassigned KAFKA-14482: - Assignee: Alexandre Dupriez > Move LogLoader to storage module >

[GitHub] [kafka] mimaison commented on pull request #12366: KAFKA-14021: Implement new KIP-618 APIs in MirrorSourceConnector

2023-02-07 Thread via GitHub
mimaison commented on PR #12366: URL: https://github.com/apache/kafka/pull/12366#issuecomment-1421014723 Thanks Chris for the quick reply! I'll take a look this week. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[jira] [Assigned] (KAFKA-14579) Move DumpLogSegments to tools

2023-02-07 Thread Alexandre Dupriez (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14579?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alexandre Dupriez reassigned KAFKA-14579: - Assignee: Alexandre Dupriez > Move DumpLogSegments to tools >

[GitHub] [kafka] C0urante commented on pull request #12366: KAFKA-14021: Implement new KIP-618 APIs in MirrorSourceConnector

2023-02-07 Thread via GitHub
C0urante commented on PR #12366: URL: https://github.com/apache/kafka/pull/12366#issuecomment-1421008599 @mimaison Thanks, I've rebased. I've also removed the changes to the offset sync semaphore since https://github.com/apache/kafka/pull/13181 should obviate them. -- This is an

[jira] [Commented] (KAFKA-14593) Move LeaderElectionCommand to tools

2023-02-07 Thread Alexandre Dupriez (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17685366#comment-17685366 ] Alexandre Dupriez commented on KAFKA-14593: ---  Thanks for following-up. I picked up one of

[GitHub] [kafka] Hangleton opened a new pull request, #13214: [KAFKA-14577] Move the scala ConsoleProducer from core to tools module

2023-02-07 Thread via GitHub
Hangleton opened a new pull request, #13214: URL: https://github.com/apache/kafka/pull/13214 Code move for the `consoleProducer` as part of KAFKA-14525 - Move CLI tools from `core` to `tools` module. The copy is as identical as possible as the original class and provides iso-functionality.

[GitHub] [kafka] jeqo commented on a diff in pull request #13177: [KAFKA-14441] Benchmark performance impact of metrics library

2023-02-07 Thread via GitHub
jeqo commented on code in PR #13177: URL: https://github.com/apache/kafka/pull/13177#discussion_r1096076032 ## jmh-benchmarks/src/main/java/org/apache/kafka/jmh/common/metrics/SensorBenchmark.java: ## @@ -0,0 +1,145 @@ +package org.apache.kafka.jmh.common.metrics; + +import

[jira] [Updated] (KAFKA-14675) Extract metadata-related tasks from Fetcher into MetadataFetcher

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14675?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk True updated KAFKA-14675: -- Reviewer: Jason Gustafson > Extract metadata-related tasks from Fetcher into MetadataFetcher >

[jira] [Commented] (KAFKA-14317) ProduceRequest timeouts are logged as network exceptions

2023-02-07 Thread Kirk True (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17685353#comment-17685353 ] Kirk True commented on KAFKA-14317: --- [~dajac] could you take another look at the PR? Thanks! >

[GitHub] [kafka] kirktrue commented on a diff in pull request #12813: KAFKA-14317: ProduceRequest timeouts are logged as network exceptions

2023-02-07 Thread via GitHub
kirktrue commented on code in PR #12813: URL: https://github.com/apache/kafka/pull/12813#discussion_r1098825578 ## clients/src/main/java/org/apache/kafka/clients/producer/internals/Sender.java: ## @@ -549,7 +549,13 @@ private boolean awaitNodeReady(Node node,

[GitHub] [kafka] divijvaidya commented on a diff in pull request #13212: MINOR: Remove accidental unnecessary code; fix comment references; use bulk operations

2023-02-07 Thread via GitHub
divijvaidya commented on code in PR #13212: URL: https://github.com/apache/kafka/pull/13212#discussion_r1098802862 ## clients/src/main/java/org/apache/kafka/clients/admin/AlterClientQuotasOptions.java: ## @@ -19,6 +19,8 @@ import

[GitHub] [kafka] mimaison commented on a diff in pull request #13195: Minor: Add JmxTool note to 3.5.0 notable changes

2023-02-07 Thread via GitHub
mimaison commented on code in PR #13195: URL: https://github.com/apache/kafka/pull/13195#discussion_r1098805966 ## docs/upgrade.html: ## @@ -19,6 +19,16 @@

[GitHub] [kafka] C0urante commented on pull request #13191: KAFKA-14060: Replace EasyMock and PowerMock with Mockito in AbstractWorkerSourceTaskTest

2023-02-07 Thread via GitHub
C0urante commented on PR #13191: URL: https://github.com/apache/kafka/pull/13191#issuecomment-1420949503 @clolov I've provided some information on that front in the description for [the ticket I linked](https://issues.apache.org/jira/browse/KAFKA-14682). TL;DR: I believe it's because we

[GitHub] [kafka] clolov commented on pull request #13191: KAFKA-14060: Replace EasyMock and PowerMock with Mockito in AbstractWorkerSourceTaskTest

2023-02-07 Thread via GitHub
clolov commented on PR #13191: URL: https://github.com/apache/kafka/pull/13191#issuecomment-1420944086 > It's also worth noting that there are unused stubbings in some of these tests, which should be failing the build but are not at the moment due to

[GitHub] [kafka] dajac commented on a diff in pull request #13196: KAFKA-14673; Add high watermark listener to Partition/Log layers

2023-02-07 Thread via GitHub
dajac commented on code in PR #13196: URL: https://github.com/apache/kafka/pull/13196#discussion_r1098775244 ## core/src/main/scala/kafka/server/ReplicaManager.scala: ## @@ -334,6 +334,29 @@ class ReplicaManager(val config: KafkaConfig, topicPartitions.foreach(tp =>

[GitHub] [kafka] clolov commented on pull request #13122: KAFKA-14594: Move LogDirsCommand to tools module

2023-02-07 Thread via GitHub
clolov commented on PR #13122: URL: https://github.com/apache/kafka/pull/13122#issuecomment-1420901116 Heya @fvaleri, I hope I have addressed all your comments. Unless I am wrong, the only difference between the two outputs is that in the old the error was ``` "error": null ```

[GitHub] [kafka] satishd commented on pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd commented on PR #13213: URL: https://github.com/apache/kafka/pull/13213#issuecomment-1420880690 Thanks @ijuma for your review. spotBugs task is fixed with the latest commit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [kafka] ijuma commented on a diff in pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
ijuma commented on code in PR #13213: URL: https://github.com/apache/kafka/pull/13213#discussion_r1098706079 ## storage/src/main/java/org/apache/kafka/storage/internals/log/AbstractIndex.java: ## @@ -64,8 +65,7 @@ private enum SearchResultType { */ private volatile

[GitHub] [kafka] satishd commented on a diff in pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd commented on code in PR #13213: URL: https://github.com/apache/kafka/pull/13213#discussion_r1098698848 ## storage/src/main/java/org/apache/kafka/storage/internals/log/AbstractIndex.java: ## @@ -64,8 +65,7 @@ private enum SearchResultType { */ private volatile

[GitHub] [kafka] satishd commented on a diff in pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd commented on code in PR #13213: URL: https://github.com/apache/kafka/pull/13213#discussion_r1098698848 ## storage/src/main/java/org/apache/kafka/storage/internals/log/AbstractIndex.java: ## @@ -64,8 +65,7 @@ private enum SearchResultType { */ private volatile

[GitHub] [kafka] satishd commented on a diff in pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd commented on code in PR #13213: URL: https://github.com/apache/kafka/pull/13213#discussion_r1098698848 ## storage/src/main/java/org/apache/kafka/storage/internals/log/AbstractIndex.java: ## @@ -64,8 +65,7 @@ private enum SearchResultType { */ private volatile

[GitHub] [kafka] dajac commented on a diff in pull request #13203: KAFKA-14048: Add new `__consumer_offsets` records from KIP-848

2023-02-07 Thread via GitHub
dajac commented on code in PR #13203: URL: https://github.com/apache/kafka/pull/13203#discussion_r1098693395 ## group-coordinator/src/main/resources/common/message/ConsumerGroupPartitionMetadataValue.json: ## @@ -0,0 +1,30 @@ +// Licensed to the Apache Software Foundation (ASF)

[GitHub] [kafka] dajac commented on a diff in pull request #13203: KAFKA-14048: Add new `__consumer_offsets` records from KIP-848

2023-02-07 Thread via GitHub
dajac commented on code in PR #13203: URL: https://github.com/apache/kafka/pull/13203#discussion_r1098691309 ## group-coordinator/src/main/resources/common/message/ConsumerGroupCurrentMemberAssignmentKey.json: ## @@ -0,0 +1,26 @@ +// Licensed to the Apache Software Foundation

[GitHub] [kafka] dajac commented on a diff in pull request #13203: KAFKA-14048: Add new `__consumer_offsets` records from KIP-848

2023-02-07 Thread via GitHub
dajac commented on code in PR #13203: URL: https://github.com/apache/kafka/pull/13203#discussion_r1098689954 ## group-coordinator/src/main/resources/common/message/ConsumerGroupCurrentMemberAssignmentKey.json: ## @@ -0,0 +1,26 @@ +// Licensed to the Apache Software Foundation

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098685000 ## core/src/test/scala/unit/kafka/server/ApiVersionManagerTest.scala: ## @@ -40,10 +40,15 @@ class ApiVersionManagerTest { listenerType = apiScope,

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098684544 ## clients/src/main/java/org/apache/kafka/common/protocol/ApiKeys.java: ## @@ -212,6 +215,36 @@ public boolean isVersionSupported(short apiVersion) { return

[GitHub] [kafka] lucasbru commented on a diff in pull request #13025: KAFKA-14299: Fix pause and resume with state updater

2023-02-07 Thread via GitHub
lucasbru commented on code in PR #13025: URL: https://github.com/apache/kafka/pull/13025#discussion_r1097298086 ## streams/src/main/java/org/apache/kafka/streams/processor/internals/TaskManager.java: ## @@ -1534,7 +1540,13 @@ Set standbyTaskIds() { Map allTasks() {

[GitHub] [kafka] ijuma commented on a diff in pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
ijuma commented on code in PR #13213: URL: https://github.com/apache/kafka/pull/13213#discussion_r1098655061 ## storage/src/main/java/org/apache/kafka/storage/internals/log/AbstractIndex.java: ## @@ -64,8 +65,7 @@ private enum SearchResultType { */ private volatile

[GitHub] [kafka] clolov commented on pull request #13212: MINOR: Remove accidental unnecessary code; fix comment references; use bulk operations

2023-02-07 Thread via GitHub
clolov commented on PR #13212: URL: https://github.com/apache/kafka/pull/13212#issuecomment-1420762600 ``` [2023-02-07T11:33:01.745Z] > Task :clients:compileJava [2023-02-07T11:33:01.745Z]

[GitHub] [kafka] urbandan commented on pull request #13163: KAFKA-14653: MirrorMakerConfig using raw properties instead of post-r…

2023-02-07 Thread via GitHub
urbandan commented on PR #13163: URL: https://github.com/apache/kafka/pull/13163#issuecomment-1420758683 @C0urante Thanks for the comments, addressed them -- 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

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098642239 ## generator/src/main/java/org/apache/kafka/message/RequestApiStabilityType.java: ## @@ -0,0 +1,27 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098641948 ## generator/src/main/java/org/apache/kafka/message/MessageSpec.java: ## @@ -70,6 +74,12 @@ public MessageSpec(@JsonProperty("name") String name,

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098634689 ## core/src/main/scala/kafka/server/ApiVersionManager.scala: ## @@ -45,55 +45,70 @@ object ApiVersionManager { listenerType, forwardingManager,

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098634292 ## clients/src/main/resources/common/message/ConsumerGroupHeartbeatResponse.json: ## @@ -0,0 +1,70 @@ +// Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098633270 ## clients/src/main/resources/common/message/ConsumerGroupHeartbeatResponse.json: ## @@ -0,0 +1,70 @@ +// Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098630635 ## clients/src/main/java/org/apache/kafka/common/protocol/ApiKeys.java: ## @@ -212,6 +215,36 @@ public boolean isVersionSupported(short apiVersion) { return

[GitHub] [kafka] dajac commented on a diff in pull request #12972: KAFKA-14391; Add ConsumerGroupHeartbeat API

2023-02-07 Thread via GitHub
dajac commented on code in PR #12972: URL: https://github.com/apache/kafka/pull/12972#discussion_r1098628288 ## clients/src/main/java/org/apache/kafka/common/errors/FencedMemberEpochException.java: ## @@ -0,0 +1,23 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [kafka] dajac commented on a diff in pull request #13116: KAFKA-14351: Controller Mutation Quota for KRaft

2023-02-07 Thread via GitHub
dajac commented on code in PR #13116: URL: https://github.com/apache/kafka/pull/13116#discussion_r1098623901 ## core/src/main/scala/kafka/server/ControllerApis.scala: ## @@ -322,15 +325,37 @@ class ControllerApis(val requestChannel: RequestChannel, } }

[GitHub] [kafka] showuon commented on pull request #13099: KAFKA-14604: avoid SASL session expiration time overflowed when calculation

2023-02-07 Thread via GitHub
showuon commented on PR #13099: URL: https://github.com/apache/kafka/pull/13099#issuecomment-1420692624 @ijuma @tombentley @clolov , I've updated the PR to throw exception when overflowed. Please take a look again. Thank you. -- This is an automated message from the Apache Git Service.

[GitHub] [kafka] showuon commented on a diff in pull request #13099: KAFKA-14604: avoid SASL session expiration time overflowed when calculation

2023-02-07 Thread via GitHub
showuon commented on code in PR #13099: URL: https://github.com/apache/kafka/pull/13099#discussion_r1098593520 ## clients/src/main/java/org/apache/kafka/common/utils/Utils.java: ## @@ -1496,4 +1496,32 @@ public static String replaceSuffix(String str, String oldSuffix, String

[GitHub] [kafka] showuon commented on a diff in pull request #13099: KAFKA-14604: avoid SASL session expiration time overflowed when calculation

2023-02-07 Thread via GitHub
showuon commented on code in PR #13099: URL: https://github.com/apache/kafka/pull/13099#discussion_r1098593071 ## clients/src/main/java/org/apache/kafka/common/utils/Utils.java: ## @@ -1496,4 +1496,32 @@ public static String replaceSuffix(String str, String oldSuffix, String

[GitHub] [kafka] satishd commented on a diff in pull request #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd commented on code in PR #13213: URL: https://github.com/apache/kafka/pull/13213#discussion_r1098558585 ## storage/src/main/java/org/apache/kafka/storage/internals/log/AbstractIndex.java: ## @@ -64,8 +65,7 @@ private enum SearchResultType { */ private volatile

[GitHub] [kafka] satishd opened a new pull request, #13213: KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread via GitHub
satishd opened a new pull request, #13213: URL: https://github.com/apache/kafka/pull/13213 KAFKA-14688 Move package org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log This is a followup item from https://github.com/apache/kafka/pull/13046 with the

[jira] [Updated] (KAFKA-14688) Move org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread Satish Duggana (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14688?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Satish Duggana updated KAFKA-14688: --- Description: This is a followup item from https://github.com/apache/kafka/pull/13046 with

[jira] [Created] (KAFKA-14688) Move org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log

2023-02-07 Thread Satish Duggana (Jira)
Satish Duggana created KAFKA-14688: -- Summary: Move org.apache.kafka.server.log.internals to org.apache.kafka.storage.internals.log Key: KAFKA-14688 URL: https://issues.apache.org/jira/browse/KAFKA-14688

  1   2   >