[GitHub] [kafka] Justinwins commented on pull request #12350: KAFKA-13856:MirrorCheckpointTask meets ConcurrentModificationException

2022-08-11 Thread GitBox
Justinwins commented on PR #12350: URL: https://github.com/apache/kafka/pull/12350#issuecomment-1212684275 @mimaison will you pls merge this simple issue ? -- 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] niket-goel opened a new pull request, #12508: KAFKA-13888: Addition of Information in DescribeQuorumResponse

2022-08-11 Thread GitBox
niket-goel opened a new pull request, #12508: URL: https://github.com/apache/kafka/pull/12508 This commit adds in the implementation for two new fields: * LastFetchTimestamp * LastCaughtUpTimestamp as they are described in KIP-836 -- This is an automated message from the Apache

[GitHub] [kafka] hachikuji merged pull request #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
hachikuji merged PR #12506: URL: https://github.com/apache/kafka/pull/12506 -- 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] [Resolved] (KAFKA-13986) DescribeQuorum does not return the observers (brokers) for the Metadata log

2022-08-11 Thread Jason Gustafson (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13986?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson resolved KAFKA-13986. - Fix Version/s: 3.3.0 Resolution: Fixed > DescribeQuorum does not return the

[jira] [Updated] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14164?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Prime Minister of Funhavistan updated KAFKA-14164: -- Description: Step 1: On a server running kafka using the

[jira] [Updated] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14164?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Prime Minister of Funhavistan updated KAFKA-14164: -- Description: Step 1: On a server running kafka using the

[jira] [Updated] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14164?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Prime Minister of Funhavistan updated KAFKA-14164: -- Description: Step 1: On a server running kafka using the

[jira] [Updated] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14164?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Prime Minister of Funhavistan updated KAFKA-14164: -- Description: Step 1: On a server running kafka using the

[jira] [Updated] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14164?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Prime Minister of Funhavistan updated KAFKA-14164: -- Description: Step 1: On a server running kafka using the

[jira] [Updated] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14164?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Prime Minister of Funhavistan updated KAFKA-14164: -- Description: Step 1: On a server running kafka using the

[GitHub] [kafka] hachikuji merged pull request #12498: KAFKA-13986; Brokers should include node.id in fetches to metadata quorum

2022-08-11 Thread GitBox
hachikuji merged PR #12498: URL: https://github.com/apache/kafka/pull/12498 -- 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] [Created] (KAFKA-14164) Enabling JMX on one Service Breaks kafka-run-class Shell Script for All

2022-08-11 Thread Prime Minister of Funhavistan (Jira)
Prime Minister of Funhavistan created KAFKA-14164: - Summary: Enabling JMX on one Service Breaks kafka-run-class Shell Script for All Key: KAFKA-14164 URL:

[jira] [Resolved] (KAFKA-14163) Build failing in streams-scala:compileScala due to zinc compiler cache

2022-08-11 Thread Jason Gustafson (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson resolved KAFKA-14163. - Resolution: Workaround > Build failing in streams-scala:compileScala due to zinc

[GitHub] [kafka] hachikuji merged pull request #12507: KAFKA-14163; Retry compilation after zinc compile cache error

2022-08-11 Thread GitBox
hachikuji merged PR #12507: URL: https://github.com/apache/kafka/pull/12507 -- 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] jsancio commented on a diff in pull request #12274: KAFKA-13959: Controller should unfence Broker with busy metadata log

2022-08-11 Thread GitBox
jsancio commented on code in PR #12274: URL: https://github.com/apache/kafka/pull/12274#discussion_r943618096 ## raft/src/main/java/org/apache/kafka/raft/internals/FuturePurgatory.java: ## @@ -56,7 +56,7 @@ CompletableFuture await(T threshold, long maxWaitTimeMs);

[GitHub] [kafka] hachikuji commented on pull request #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
hachikuji commented on PR #12506: URL: https://github.com/apache/kafka/pull/12506#issuecomment-1212307242 @jsancio Yeah, let's use the same jira for the kraft issue if that's ok with you. Let me see if I can get a PR out today. -- This is an automated message from the Apache Git Service.

[GitHub] [kafka] hachikuji opened a new pull request, #12507: KAFKA-14163; Retry compilation after zinc compile cache error

2022-08-11 Thread GitBox
hachikuji opened a new pull request, #12507: URL: https://github.com/apache/kafka/pull/12507 Borrowing this workaround from @lbradstreet for the same problem. Co-authored-by: Lucas Bradstreet ### Committer Checklist (excluded from commit message) - [ ] Verify design and

[jira] [Updated] (KAFKA-14163) Build failing in streams-scala:compileScala due to zinc compiler cache

2022-08-11 Thread Jason Gustafson (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson updated KAFKA-14163: Summary: Build failing in streams-scala:compileScala due to zinc compiler cache (was:

[GitHub] [kafka] dajac commented on a diff in pull request #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
dajac commented on code in PR #12506: URL: https://github.com/apache/kafka/pull/12506#discussion_r943741675 ## core/src/main/scala/kafka/controller/KafkaController.scala: ## @@ -2336,7 +2336,14 @@ class KafkaController(val config: KafkaConfig,

[jira] [Updated] (KAFKA-14163) Build failing in scala:compileScala due to zinc compiler cache

2022-08-11 Thread Jason Gustafson (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson updated KAFKA-14163: Description: I have been seeing builds failing recently with the following error:  

[jira] [Updated] (KAFKA-14163) Build failing in scala:compileScala due to zinc compiler cache

2022-08-11 Thread Jason Gustafson (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson updated KAFKA-14163: Description: I have been seeing builds failing recently with the following error:

[jira] [Created] (KAFKA-14163) Build failing in scala:compileScala due to zinc compiler cache

2022-08-11 Thread Jason Gustafson (Jira)
Jason Gustafson created KAFKA-14163: --- Summary: Build failing in scala:compileScala due to zinc compiler cache Key: KAFKA-14163 URL: https://issues.apache.org/jira/browse/KAFKA-14163 Project: Kafka

[GitHub] [kafka] jsancio commented on pull request #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
jsancio commented on PR #12506: URL: https://github.com/apache/kafka/pull/12506#issuecomment-1212271060 > but I believe the problem exists for kraft as well. Are you planning to use the same Jira (KAFKA-14154 to track this? I assume that the KRaft fix is also a blocker for 3.3.0.

[GitHub] [kafka] hachikuji commented on pull request #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
hachikuji commented on PR #12506: URL: https://github.com/apache/kafka/pull/12506#issuecomment-1212257796 One note: this patch fixes the problem for the zk controller, but I believe the problem exists for kraft as well. I am thinking we can address that separately with a slightly different

[GitHub] [kafka] hachikuji commented on pull request #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
hachikuji commented on PR #12506: URL: https://github.com/apache/kafka/pull/12506#issuecomment-1212250619 cc @jsancio @dajac -- 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] hachikuji opened a new pull request, #12506: KAFKA-14154; Return NOT_CONTROLLER from AlterPartition if leader is ahead of controller

2022-08-11 Thread GitBox
hachikuji opened a new pull request, #12506: URL: https://github.com/apache/kafka/pull/12506 It is possible for the leader to send an `AlterPartition` request to a zombie controller which includes either a partition or leader epoch which is larger than what is found in the controller

[GitHub] [kafka] hachikuji closed pull request #12499: KAFKA-14154; Ensure AlterPartition not sent to stale controller

2022-08-11 Thread GitBox
hachikuji closed pull request #12499: KAFKA-14154; Ensure AlterPartition not sent to stale controller URL: https://github.com/apache/kafka/pull/12499 -- 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] hachikuji commented on pull request #12499: KAFKA-14154; Ensure AlterPartition not sent to stale controller

2022-08-11 Thread GitBox
hachikuji commented on PR #12499: URL: https://github.com/apache/kafka/pull/12499#issuecomment-1212226476 I am going to close this PR. On the one hand, it does not address the problem for KRaft; on the other, we have thought of a simpler fix for the zk controller, which I will open

[GitHub] [kafka] mimaison commented on a diff in pull request #12046: KAFKA-10360: Allow disabling JMX Reporter (KIP-830)

2022-08-11 Thread GitBox
mimaison commented on code in PR #12046: URL: https://github.com/apache/kafka/pull/12046#discussion_r943688586 ## server-common/src/main/java/org/apache/kafka/server/metrics/KafkaYammerMetrics.java: ## @@ -53,16 +72,21 @@ public static MetricsRegistry defaultRegistry() { }

[GitHub] [kafka] yashmayya commented on a diff in pull request #12490: KAFKA-14147: Prevent deferredTaskUpdates map from growing monotonically in KafkaConfigBackingStore

2022-08-11 Thread GitBox
yashmayya commented on code in PR #12490: URL: https://github.com/apache/kafka/pull/12490#discussion_r943666445 ## connect/runtime/src/main/java/org/apache/kafka/connect/storage/KafkaConfigBackingStore.java: ## @@ -853,6 +853,9 @@ private void

[jira] [Updated] (KAFKA-14133) Remaining EasyMock to Mockito tests

2022-08-11 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-14133: -- Description: {color:#DE350B}There are tests which use both PowerMock and EasyMock. I have

[GitHub] [kafka] clolov commented on pull request #12505: KAFKA-14133: Replace EasyMock with Mockito in streams tests

2022-08-11 Thread GitBox
clolov commented on PR #12505: URL: https://github.com/apache/kafka/pull/12505#issuecomment-1212166713 Tagging @mdedetrich @cadonna @ijuma for visibility and reviews :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [kafka] clolov commented on a diff in pull request #12505: KAFKA-14133: Replace EasyMock with Mockito in streams tests

2022-08-11 Thread GitBox
clolov commented on code in PR #12505: URL: https://github.com/apache/kafka/pull/12505#discussion_r943639431 ## streams/src/test/java/org/apache/kafka/streams/state/internals/KeyValueStoreBuilderTest.java: ## @@ -126,27 +121,15 @@ public void

[GitHub] [kafka] clolov commented on a diff in pull request #12505: KAFKA-14133: Replace EasyMock with Mockito in streams tests

2022-08-11 Thread GitBox
clolov commented on code in PR #12505: URL: https://github.com/apache/kafka/pull/12505#discussion_r943639431 ## streams/src/test/java/org/apache/kafka/streams/state/internals/KeyValueStoreBuilderTest.java: ## @@ -126,27 +121,15 @@ public void

[GitHub] [kafka] clolov opened a new pull request, #12505: KAFKA-14133: Replace EasyMock with Mockito in streams tests

2022-08-11 Thread GitBox
clolov opened a new pull request, #12505: URL: https://github.com/apache/kafka/pull/12505 Batch 2 of the tests detailed in https://issues.apache.org/jira/browse/KAFKA-14133 which use EasyMock and need to be moved to Mockito. -- This is an automated message from the Apache Git Service.

[jira] [Assigned] (KAFKA-14134) Replace EasyMock with Mockito for WorkerConnectorTest

2022-08-11 Thread Yash Mayya (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14134?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yash Mayya reassigned KAFKA-14134: -- Assignee: Yash Mayya > Replace EasyMock with Mockito for WorkerConnectorTest >

[jira] [Assigned] (KAFKA-14147) Some map objects in KafkaConfigBackingStore grow in size monotonically

2022-08-11 Thread Yash Mayya (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yash Mayya reassigned KAFKA-14147: -- Assignee: Yash Mayya > Some map objects in KafkaConfigBackingStore grow in size

[jira] [Updated] (KAFKA-14133) Remaining EasyMock to Mockito tests

2022-08-11 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-14133: -- Description: {color:#DE350B}There are tests which use both PowerMock and EasyMock. I have

[GitHub] [kafka] mimaison opened a new pull request, #12504: KAFKA-14149: Fix DynamicBrokerReconfigurationTest in 3.2

2022-08-11 Thread GitBox
mimaison opened a new pull request, #12504: URL: https://github.com/apache/kafka/pull/12504 @mumrah @cmccabe It looks like a way to fix `DynamicBrokerReconfigurationTest` in 3.2 is to backport https://github.com/apache/kafka/commit/9c3f605fc78f297ecf5accdcdec18471c19cf7d6. I think it

[jira] [Assigned] (KAFKA-14162) HoistField SMT should not return an immutable map for schemaless key/value

2022-08-11 Thread Yash Mayya (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14162?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yash Mayya reassigned KAFKA-14162: -- Assignee: Yash Mayya > HoistField SMT should not return an immutable map for schemaless

[GitHub] [kafka] jsancio commented on a diff in pull request #12274: KAFKA-13959: Controller should unfence Broker with busy metadata log

2022-08-11 Thread GitBox
jsancio commented on code in PR #12274: URL: https://github.com/apache/kafka/pull/12274#discussion_r943606596 ## metadata/src/main/java/org/apache/kafka/controller/ReplicationControlManager.java: ## @@ -1382,7 +1383,6 @@ ControllerResult processBrokerHeartbeat(

[GitHub] [kafka] jsancio commented on pull request #12274: KAFKA-13959: Controller should unfence Broker with busy metadata log

2022-08-11 Thread GitBox
jsancio commented on PR #12274: URL: https://github.com/apache/kafka/pull/12274#issuecomment-1212113291 @dengziming take a look at the tests failures. All of the failures in `ClusterControlManagerTest` seems related to this change. -- This is an automated message from the Apache Git

[jira] [Updated] (KAFKA-14133) Remaining EasyMock to Mockito tests

2022-08-11 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-14133: -- Description: {color:#DE350B}There are tests which use both PowerMock and EasyMock. I have

[GitHub] [kafka] chia7712 opened a new pull request, #12503: Appending value to LIST config should not generate empty string with …

2022-08-11 Thread GitBox
chia7712 opened a new pull request, #12503: URL: https://github.com/apache/kafka/pull/12503 We noticed this issue when appending value to `leader.replication.throttled.replicas`. The default value is empty string, so it produces `,0:0` rather than `0:0`. For another, parsing throttled

[jira] [Updated] (KAFKA-14133) Remaining EasyMock to Mockito tests

2022-08-11 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-14133: -- Description: {color:#DE350B}There are tests which use both PowerMock and EasyMock. I have

[jira] [Commented] (KAFKA-14098) Internal Kafka clients used by Kafka Connect should have distinguishable client IDs

2022-08-11 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17578486#comment-17578486 ] Mickael Maison commented on KAFKA-14098: This would be very valuable. When looking at metrics

[GitHub] [kafka] dstelljes commented on pull request #12295: KAFKA-13586: Prevent exception thrown during connector update from crashing distributed herder

2022-08-11 Thread GitBox
dstelljes commented on PR #12295: URL: https://github.com/apache/kafka/pull/12295#issuecomment-1212026332 Sorry for the wait on this—it looks like work to switch the test to Mockito is in flight now, so I’ll hang tight for now and rebase once that lands. I do have an account on the

[jira] [Updated] (KAFKA-14133) Remaining EasyMock to Mockito tests

2022-08-11 Thread Christo Lolov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christo Lolov updated KAFKA-14133: -- Description: {color:#DE350B}There are tests which use both PowerMock and EasyMock. I have

[GitHub] [kafka] andymg3 commented on a diff in pull request #12499: KAFKA-14154; Ensure AlterPartition not sent to stale controller

2022-08-11 Thread GitBox
andymg3 commented on code in PR #12499: URL: https://github.com/apache/kafka/pull/12499#discussion_r941967874 ## core/src/main/scala/kafka/server/BrokerToControllerChannelManager.scala: ## @@ -364,15 +423,15 @@ class BrokerToControllerRequestThread( } override def

[jira] [Resolved] (KAFKA-14161) kafka-consumer-group.sh --list not list all consumer groups

2022-08-11 Thread Luke Chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14161?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Luke Chen resolved KAFKA-14161. --- Resolution: Not A Problem > kafka-consumer-group.sh --list not list all consumer groups >

[jira] [Commented] (KAFKA-14161) kafka-consumer-group.sh --list not list all consumer groups

2022-08-11 Thread Luke Chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17578351#comment-17578351 ] Luke Chen commented on KAFKA-14161: --- [~yandufeng] , you should check your consumer setting. Not every

[GitHub] [kafka] tombentley commented on a diff in pull request #12046: KAFKA-10360: Allow disabling JMX Reporter (KIP-830)

2022-08-11 Thread GitBox
tombentley commented on code in PR #12046: URL: https://github.com/apache/kafka/pull/12046#discussion_r943247605 ## server-common/src/main/java/org/apache/kafka/server/metrics/KafkaYammerMetrics.java: ## @@ -53,16 +72,21 @@ public static MetricsRegistry defaultRegistry() {

[GitHub] [kafka] ashmeet13 commented on a diff in pull request #12414: KAFKA-14073 Logging the reason for Snapshot

2022-08-11 Thread GitBox
ashmeet13 commented on code in PR #12414: URL: https://github.com/apache/kafka/pull/12414#discussion_r943241275 ## raft/src/main/java/org/apache/kafka/raft/ReplicatedCounter.java: ## @@ -107,6 +107,8 @@ public synchronized void handleCommit(BatchReader reader) { }

[GitHub] [kafka] ashmeet13 commented on a diff in pull request #12414: KAFKA-14073 Logging the reason for Snapshot

2022-08-11 Thread GitBox
ashmeet13 commented on code in PR #12414: URL: https://github.com/apache/kafka/pull/12414#discussion_r943240735 ## core/src/main/scala/kafka/server/metadata/BrokerMetadataSnapshotter.scala: ## @@ -59,11 +60,19 @@ class BrokerMetadataSnapshotter( val writer =

[GitHub] [kafka] tombentley commented on a diff in pull request #12046: KAFKA-10360: Allow disabling JMX Reporter (KIP-830)

2022-08-11 Thread GitBox
tombentley commented on code in PR #12046: URL: https://github.com/apache/kafka/pull/12046#discussion_r943234303 ## server-common/src/main/java/org/apache/kafka/server/metrics/KafkaYammerMetrics.java: ## @@ -53,16 +72,21 @@ public static MetricsRegistry defaultRegistry() {

[GitHub] [kafka] yashmayya commented on pull request #12502: KAFKA-14162: HoistField SMT should not return an immutable map for schemaless key/value

2022-08-11 Thread GitBox
yashmayya commented on PR #12502: URL: https://github.com/apache/kafka/pull/12502#issuecomment-1211696790 @C0urante another small PR for you to review whenever you get a chance? Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [kafka] yashmayya opened a new pull request, #12502: KAFKA-14162: HoistField SMT should not return an immutable map for schemaless key/value

2022-08-11 Thread GitBox
yashmayya opened a new pull request, #12502: URL: https://github.com/apache/kafka/pull/12502 - https://issues.apache.org/jira/browse/KAFKA-14162 - The HoistField SMT currently returns an immutable map for schemaless keys and values -

[jira] [Created] (KAFKA-14162) HoistField SMT should not return an immutable map for schemaless key/value

2022-08-11 Thread Yash Mayya (Jira)
Yash Mayya created KAFKA-14162: -- Summary: HoistField SMT should not return an immutable map for schemaless key/value Key: KAFKA-14162 URL: https://issues.apache.org/jira/browse/KAFKA-14162 Project:

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

2022-08-11 Thread GitBox
showuon commented on code in PR #12392: URL: https://github.com/apache/kafka/pull/12392#discussion_r943219606 ## clients/src/test/java/org/apache/kafka/clients/producer/internals/TransactionManagerTest.java: ## @@ -2594,27 +2596,20 @@ public void testDropCommitOnBatchExpiry()

[GitHub] [kafka] showuon commented on pull request #12500: MINOR: Remove spurious sleep in ConsumerCoordinatorTest

2022-08-11 Thread GitBox
showuon commented on PR #12500: URL: https://github.com/apache/kafka/pull/12500#issuecomment-1211653244 Triggering another jenkins build to make sure it passes the tests. https://ci-builds.apache.org/job/Kafka/job/kafka-pr/view/change-requests/job/PR-12500/2/ -- This is an automated

[jira] [Created] (KAFKA-14161) kafka-consumer-group.sh --list not list all consumer groups

2022-08-11 Thread yandufeng (Jira)
yandufeng created KAFKA-14161: - Summary: kafka-consumer-group.sh --list not list all consumer groups Key: KAFKA-14161 URL: https://issues.apache.org/jira/browse/KAFKA-14161 Project: Kafka Issue

[GitHub] [kafka] dengziming commented on a diff in pull request #12274: KAFKA-13959: Controller should unfence Broker with busy metadata log

2022-08-11 Thread GitBox
dengziming commented on code in PR #12274: URL: https://github.com/apache/kafka/pull/12274#discussion_r943173917 ## metadata/src/main/java/org/apache/kafka/controller/QuorumController.java: ## @@ -1305,12 +1305,11 @@ private void handleFeatureControlChange() { } }

[GitHub] [kafka] clolov commented on pull request #11792: Replace EasyMock/PowerMock with Mockito in DistributedHerderTest

2022-08-11 Thread GitBox
clolov commented on PR #11792: URL: https://github.com/apache/kafka/pull/11792#issuecomment-1211589682 Okay @dplavcic! Thanks both for picking it up and for letting us know :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [kafka] ahuang98 commented on a diff in pull request #12479: MINOR; Convert some integration tests to run with the KRaft modes

2022-08-11 Thread GitBox
ahuang98 commented on code in PR #12479: URL: https://github.com/apache/kafka/pull/12479#discussion_r943125682 ## core/src/test/scala/unit/kafka/admin/DeleteTopicTest.scala: ## @@ -45,103 +59,122 @@ class DeleteTopicTest extends QuorumTestHarness { @AfterEach override