[GitHub] [kafka] cmccabe commented on pull request #12695: KAFKA-14265: Prefix ACLs may shadow other prefix ACLs

2022-09-28 Thread GitBox
cmccabe commented on PR #12695: URL: https://github.com/apache/kafka/pull/12695#issuecomment-1261804450 I added two tests. One unit test and one integration test. I verified that both fail without the fix. The integration tests succeeds with AclAuthorizer prior to the StandardAuthorizer fix

[GitHub] [kafka] hachikuji commented on a diff in pull request #12695: KAFKA-14265: Prefix ACLs may shadow other prefix ACLs

2022-09-28 Thread GitBox
hachikuji commented on code in PR #12695: URL: https://github.com/apache/kafka/pull/12695#discussion_r982972020 ## metadata/src/main/java/org/apache/kafka/metadata/authorizer/StandardAuthorizerData.java: ## @@ -394,35 +399,61 @@ private MatchingAclRule findAclRule( retu

[GitHub] [kafka] wykkevin closed pull request #12696: Insert ctest config in scala

2022-09-28 Thread GitBox
wykkevin closed pull request #12696: Insert ctest config in scala URL: https://github.com/apache/kafka/pull/12696 -- 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

[jira] [Created] (KAFKA-14265) Prefix ACLs may shadow other prefix ACLs

2022-09-28 Thread Colin McCabe (Jira)
Colin McCabe created KAFKA-14265: Summary: Prefix ACLs may shadow other prefix ACLs Key: KAFKA-14265 URL: https://issues.apache.org/jira/browse/KAFKA-14265 Project: Kafka Issue Type: Bug

[jira] [Commented] (KAFKA-2526) Console Producer / Consumer's serde config is not working

2022-09-28 Thread Alex Collins (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-2526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610748#comment-17610748 ] Alex Collins commented on KAFKA-2526: - I've created a PR to fix this.  [https://gith

[GitHub] [kafka] guozhangwang merged pull request #12659: KAFKA-10199: Integrate Topology Pause/Resume with StateUpdater

2022-09-28 Thread GitBox
guozhangwang merged PR #12659: URL: https://github.com/apache/kafka/pull/12659 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.ap

[GitHub] [kafka] hachikuji commented on a diff in pull request #12634: KAFKA-14225: Fix deadlock caused by lazy val exemptSensor

2022-09-28 Thread GitBox
hachikuji commented on code in PR #12634: URL: https://github.com/apache/kafka/pull/12634#discussion_r982935129 ## core/src/main/scala/kafka/server/ClientRequestQuotaManager.scala: ## @@ -30,6 +30,9 @@ import scala.jdk.CollectionConverters._ object ClientRequestQuotaManager {

[GitHub] [kafka] alexec opened a new pull request, #12694: fix: deserializer/serializer from properties for console consume/producer. Fixes KAFKA-2526

2022-09-28 Thread GitBox
alexec opened a new pull request, #12694: URL: https://github.com/apache/kafka/pull/12694 Signed-off-by: Alex Collins *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

[GitHub] [kafka] hshi2022 commented on pull request #12634: KAFKA-14225: Fix deadlock caused by lazy val exemptSensor

2022-09-28 Thread GitBox
hshi2022 commented on PR #12634: URL: https://github.com/apache/kafka/pull/12634#issuecomment-1261498478 @hachikuji Thanks for review. I have addressed your comments. And I am wondering whether you know how to rerun the checks/tests without submitting any changes. This PR currently fails a

[GitHub] [kafka] hshi2022 commented on a diff in pull request #12634: KAFKA-14225: Fix deadlock caused by lazy val exemptSensor

2022-09-28 Thread GitBox
hshi2022 commented on code in PR #12634: URL: https://github.com/apache/kafka/pull/12634#discussion_r982817712 ## core/src/main/scala/kafka/server/ClientRequestQuotaManager.scala: ## @@ -30,6 +30,9 @@ import scala.jdk.CollectionConverters._ object ClientRequestQuotaManager {

[jira] [Updated] (KAFKA-13657) StandardAuthorizer should implement the early start listener logic described in KIP-801

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-13657: --- Fix Version/s: (was: 3.3) > StandardAuthorizer should implement

[jira] [Updated] (KAFKA-13206) shutting down broker needs to stop fetching as a follower in KRaft mode

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13206?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-13206: --- Fix Version/s: (was: 3.3) > shutting down broker needs to stop f

[jira] [Resolved] (KAFKA-14259) BrokerRegistration#toString throws an exception, terminating metadata replay

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio resolved KAFKA-14259. Resolution: Fixed > BrokerRegistration#toString throws an exceptio

[jira] [Updated] (KAFKA-13806) Check CRC when reading snapshots

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13806?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-13806: --- Fix Version/s: (was: 3.3) > Check CRC when reading snapshots > -

[jira] [Updated] (KAFKA-14259) BrokerRegistration#toString throws an exception, terminating metadata replay

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-14259: --- Fix Version/s: 3.3.0 (was: 3.3) > BrokerRegis

[jira] [Updated] (KAFKA-14259) BrokerRegistration#toString throws an exception, terminating metadata replay

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-14259: --- Affects Version/s: (was: 3.3) > BrokerRegistration#toString thro

[GitHub] [kafka] see-quick commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
see-quick commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982830205 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return elec

[GitHub] [kafka] philipnee commented on a diff in pull request #12663: KAFKA-14247: Define event handler interface and events

2022-09-28 Thread GitBox
philipnee commented on code in PR #12663: URL: https://github.com/apache/kafka/pull/12663#discussion_r982820685 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractPrototypeAsyncConsumer.java: ## @@ -0,0 +1,119 @@ +/* + * Licensed to the Apache Software

[GitHub] [kafka] guozhangwang commented on pull request #12683: KAFKA-4852: Fix ByteBufferSerializer#serialize(String, ByteBuffer) not compatible with offsets

2022-09-28 Thread GitBox
guozhangwang commented on PR #12683: URL: https://github.com/apache/kafka/pull/12683#issuecomment-1261408099 I re-run the test case in trunk as well as in your branch locally with IDE, and both passes, so I think it's due to jenkins machines compute resource limit itself. But I got a

[GitHub] [kafka] philipnee commented on a diff in pull request #12663: KAFKA-14247: Define event handler interface and events

2022-09-28 Thread GitBox
philipnee commented on code in PR #12663: URL: https://github.com/apache/kafka/pull/12663#discussion_r982803945 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractPrototypeAsyncConsumer.java: ## @@ -0,0 +1,119 @@ +/* + * Licensed to the Apache Software

[GitHub] [kafka] hachikuji commented on a diff in pull request #12663: KAFKA-14247: Define event handler interface and events

2022-09-28 Thread GitBox
hachikuji commented on code in PR #12663: URL: https://github.com/apache/kafka/pull/12663#discussion_r982794220 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractPrototypeAsyncConsumer.java: ## @@ -0,0 +1,119 @@ +/* + * Licensed to the Apache Software

[GitHub] [kafka] divijvaidya commented on a diff in pull request #12449: KAFKA-12947: Replace EasyMock and PowerMock with Mockito for StreamsMetricsImplTest

2022-09-28 Thread GitBox
divijvaidya commented on code in PR #12449: URL: https://github.com/apache/kafka/pull/12449#discussion_r982716173 ## streams/src/test/java/org/apache/kafka/streams/processor/internals/metrics/StreamsMetricsImplTest.java: ## @@ -452,25 +441,24 @@ public void shouldNotUseSameStor

[GitHub] [kafka] hachikuji commented on pull request #12669: KAFKA-14248: Fix flaky test PlaintextAdminIntegrationTest.testCreateTopicsReturnsConfigs

2022-09-28 Thread GitBox
hachikuji commented on PR #12669: URL: https://github.com/apache/kafka/pull/12669#issuecomment-1261291916 I started one more build since the last one had a bunch of failures due to a lingering thread. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [kafka] divijvaidya commented on a diff in pull request #12229: MINOR: Include the inner exception stack trace when re-throwing an exception

2022-09-28 Thread GitBox
divijvaidya commented on code in PR #12229: URL: https://github.com/apache/kafka/pull/12229#discussion_r982709687 ## clients/src/main/java/org/apache/kafka/common/protocol/types/TaggedFields.java: ## @@ -165,7 +165,7 @@ public Map validate(Object item) { }

[GitHub] [kafka] divijvaidya commented on a diff in pull request #12590: KAFKA-7109: Close fetch sessions on close of consumer

2022-09-28 Thread GitBox
divijvaidya commented on code in PR #12590: URL: https://github.com/apache/kafka/pull/12590#discussion_r982695511 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/Fetcher.java: ## @@ -1933,11 +1941,79 @@ private Map topicPartitionTags(TopicPartition tp) {

[GitHub] [kafka] jolshan commented on pull request #12392: KAFKA-14053: Transactional producer should bump the epoch and skip ab…

2022-09-28 Thread GitBox
jolshan commented on PR #12392: URL: https://github.com/apache/kafka/pull/12392#issuecomment-1261212204 @urbandan > If it is better to keep the producer in a usable state, I can give it a shot. I had one experiment in which I tried keeping the producer usable by increasing the epoch on

[jira] [Updated] (KAFKA-13888) KIP-836: Expose replication information of the cluster metadata

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13888?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-13888: --- Summary: KIP-836: Expose replication information of the cluster metad

[GitHub] [kafka] vamossagar12 commented on a diff in pull request #12561: KAFKA-12495: Exponential backoff retry to prevent rebalance storms when worker joins after revoking rebalance

2022-09-28 Thread GitBox
vamossagar12 commented on code in PR #12561: URL: https://github.com/apache/kafka/pull/12561#discussion_r982661269 ## connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/WorkerCoordinatorIncrementalTest.java: ## @@ -517,13 +517,13 @@ public void testTaskAs

[jira] [Updated] (KAFKA-13410) KRaft to KRaft Upgrades

2022-09-28 Thread Jose Armando Garcia Sancio (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jose Armando Garcia Sancio updated KAFKA-13410: --- Summary: KRaft to KRaft Upgrades (was: KRaft Upgrades) > KRaft to K

[GitHub] [kafka] C0urante commented on pull request #12671: KAFKA-14250: MirrorSourceTask exception causes the task to fail

2022-09-28 Thread GitBox
C0urante commented on PR #12671: URL: https://github.com/apache/kafka/pull/12671#issuecomment-1261137537 That's interesting--were you running a dedicated MM2 cluster, or were you deploying MM2 as a connector on a vanilla Kafka Connect cluster? And if the latter, were you able to read the st

[GitHub] [kafka] viktorsomogyi commented on pull request #12671: KAFKA-14250: MirrorSourceTask exception causes the task to fail

2022-09-28 Thread GitBox
viktorsomogyi commented on PR #12671: URL: https://github.com/apache/kafka/pull/12671#issuecomment-1261125559 To me it seems like it persists. Let me explain what I see in my logs, maybe I'm in the wrong here. So first let's start when it stops the task during the frequent rebalances:

[GitHub] [kafka] divijvaidya commented on a diff in pull request #11390: [KAFKA-13369] Follower fetch protocol changes for tiered storage.

2022-09-28 Thread GitBox
divijvaidya commented on code in PR #11390: URL: https://github.com/apache/kafka/pull/11390#discussion_r898097965 ## core/src/main/scala/kafka/log/remote/RemoteLogManager.scala: ## @@ -0,0 +1,291 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [kafka] see-quick commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
see-quick commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982473889 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return elec

[GitHub] [kafka] ijuma commented on a diff in pull request #12590: KAFKA-7109: Close fetch sessions on close of consumer

2022-09-28 Thread GitBox
ijuma commented on code in PR #12590: URL: https://github.com/apache/kafka/pull/12590#discussion_r982463816 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/Fetcher.java: ## @@ -1933,11 +1941,79 @@ private Map topicPartitionTags(TopicPartition tp) {

[GitHub] [kafka] ijuma commented on pull request #12688: MINOR: Delete config/kraft/README.me

2022-09-28 Thread GitBox
ijuma commented on PR #12688: URL: https://github.com/apache/kafka/pull/12688#issuecomment-1260973649 @showuon Yes, I know, but I still think we should delete it since we now have the documentation in the right place. This was a placeholder until we had that. -- This is an automated mess

[GitHub] [kafka] mimaison commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
mimaison commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982449921 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return elect

[GitHub] [kafka] see-quick commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
see-quick commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982443619 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return elec

[GitHub] [kafka] ijuma commented on a diff in pull request #12229: MINOR: Include the inner exception stack trace when re-throwing an exception

2022-09-28 Thread GitBox
ijuma commented on code in PR #12229: URL: https://github.com/apache/kafka/pull/12229#discussion_r982437499 ## clients/src/main/java/org/apache/kafka/common/protocol/types/TaggedFields.java: ## @@ -165,7 +165,7 @@ public Map validate(Object item) { } re

[GitHub] [kafka] divijvaidya commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
divijvaidya commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982359239 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return el

[GitHub] [kafka] mimaison commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
mimaison commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982348035 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return elect

[jira] [Updated] (KAFKA-10733) Enhance Transactional Producer Exception Handling

2022-09-28 Thread Lucas Brutschy (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10733?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lucas Brutschy updated KAFKA-10733: --- Summary: Enhance Transactional Producer Exception Handling (was: Enforce exception thrown f

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

2022-09-28 Thread jianbin.chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] jianbin.chen closed KAFKA-14257. > Unexpected error INCONSISTENT_CLUSTER_ID in VOTE response >

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

2022-09-28 Thread jianbin.chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] jianbin.chen resolved KAFKA-14257. -- Resolution: Done > Unexpected error INCONSISTENT_CLUSTER_ID in VOTE response > ---

[GitHub] [kafka] see-quick commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
see-quick commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982190906 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return elec

[jira] [Comment Edited] (KAFKA-3332) Consumer can't consume messages from zookeeper chroot

2022-09-28 Thread Hang HOU (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-3332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610459#comment-17610459 ] Hang HOU edited comment on KAFKA-3332 at 9/28/22 9:19 AM: -- About

[jira] [Commented] (KAFKA-3332) Consumer can't consume messages from zookeeper chroot

2022-09-28 Thread Hang HOU (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-3332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610459#comment-17610459 ] Hang HOU commented on KAFKA-3332: - About my last description,I found it was my mistake. I

[GitHub] [kafka] divijvaidya commented on a diff in pull request #12692: MINOR: Add test cases to the Raft module

2022-09-28 Thread GitBox
divijvaidya commented on code in PR #12692: URL: https://github.com/apache/kafka/pull/12692#discussion_r982147666 ## raft/src/main/java/org/apache/kafka/raft/VotedState.java: ## @@ -92,11 +92,6 @@ public boolean hasElectionTimeoutExpired(long currentTimeMs) { return el