Jenkins build is still unstable: Kafka » Kafka Branch Builder » trunk #644

2022-01-28 Thread Apache Jenkins Server
See

[jira] [Created] (KAFKA-13625) KIP-817: Fix inconsistency in dynamic application log levels

2022-01-28 Thread Dongjin Lee (Jira)
Dongjin Lee created KAFKA-13625: --- Summary: KIP-817: Fix inconsistency in dynamic application log levels Key: KAFKA-13625 URL: https://issues.apache.org/jira/browse/KAFKA-13625 Project: Kafka I

[DISCUSS] KIP-817: Fix inconsistency in dynamic application log levels

2022-01-28 Thread Dongjin Lee
Hi Kafka dev, I would like to start the discussion of KIP-817: Fix inconsistency in dynamic application log levels. https://cwiki.apache.org/confluence/display/KAFKA/KIP-817%3A+Fix+inconsistency+in+dynamic+application+log+levels This is rather a minor issue, but I found it while working with KIP

[GitHub] [kafka-site] Venom-git opened a new pull request #395: Update powered-by.html

2022-01-28 Thread GitBox
Venom-git opened a new pull request #395: URL: https://github.com/apache/kafka-site/pull/395 Please help to update our organization as well , as we are using #Apache Kafka in our environment. ![morningstar](https://user-images.githubusercontent.com/59806389/151538599-fa7cc1bc-f299-

Jenkins build is still unstable: Kafka » Kafka Branch Builder » trunk #645

2022-01-28 Thread Apache Jenkins Server
See

Re: [DISCUSS] KIP-814: Static membership protocol should let the leader skip assignment

2022-01-28 Thread David Jacot
Hi all, If there are no further comments. I plan to start a vote thread on Monday. Best, David On Wed, Jan 26, 2022 at 3:16 PM David Jacot wrote: > > Hi Hector, > > Thanks for your comments. > > The idea is to add a flag to the onJoinLeader leader method. We > must do this because the ConsumerC

Re: [DISCUSS] KIP-814: Static membership protocol should let the leader skip assignment

2022-01-28 Thread Luke Chen
Hi David, Thanks for the KIP! The solution makes sense to me. One thing to clarify. You mentioned in the KIP that: > (After leader rejoin) Then the leader will send an *empty SyncGroup* to collect its own (and existing) assignment. I'd like to confirm that the *empty SyncGroup* is meaning the `

[jira] [Created] (KAFKA-13626) NullPointerException in Selector.pollSelectionKeys: channel is null

2022-01-28 Thread Jira
Daniel Häuser created KAFKA-13626: - Summary: NullPointerException in Selector.pollSelectionKeys: channel is null Key: KAFKA-13626 URL: https://issues.apache.org/jira/browse/KAFKA-13626 Project: Kafka

Re: [DISCUSS] KIP-653: Upgrade log4j to log4j2

2022-01-28 Thread Mickael Maison
Hi Dongjin, Before starting to review the PR, I was doing a bit of research of the topic and I noticed ZooKeeper actually depends on log4j1. In zookeeper-server, by default a few JMX beans are registered for log4j1. When starting ZooKeeper you get a log line like: INFO Log4j 1.2 jmx support found

[jira] [Created] (KAFKA-13627) Topology changes shouldn't require a full reset of local state

2022-01-28 Thread Nicholas Telford (Jira)
Nicholas Telford created KAFKA-13627: Summary: Topology changes shouldn't require a full reset of local state Key: KAFKA-13627 URL: https://issues.apache.org/jira/browse/KAFKA-13627 Project: Kafka

Re: [DISCUSS] KIP-814: Static membership protocol should let the leader skip assignment

2022-01-28 Thread David Jacot
Hi Luke, That's right. I have clarified this in the KIP. Thanks, David On Fri, Jan 28, 2022 at 2:23 PM Luke Chen wrote: > > Hi David, > > Thanks for the KIP! > The solution makes sense to me. > > One thing to clarify. You mentioned in the KIP that: > > > (After leader rejoin) Then the leader wi

Re: KIP-769: Connect API to retrieve connector configuration definitions

2022-01-28 Thread Mickael Maison
Thanks Chris and Tom for the feedback. I'm having a hard time making a decision on this point! Re-reading your previous comments, I agree that we're lacking a contract on the version field, and we can see value (supporting multiple versions for example) if this was improved. But I'm starting to w

Re: [VOTE] KIP-792: Add "generation" field into consumer protocol

2022-01-28 Thread Tom Bentley
Hi Luke, Thanks for the KIP, +1 (binding). Kind regards, Tom On Wed, 19 Jan 2022 at 13:16, Luke Chen wrote: > Hi all, > > Bump this thread to see if there are other comments to this KIP. > So far, I have one +1 vote (binding), and need more votes. > > Thank you. > Luke > > On Tue, Dec 21, 202

Re: KIP-769: Connect API to retrieve connector configuration definitions

2022-01-28 Thread Chris Egerton
I think a simple approach here is fine. +1 for removing versioning for other plugins. On Fri, Jan 28, 2022 at 11:17 AM Mickael Maison wrote: > Thanks Chris and Tom for the feedback. > > I'm having a hard time making a decision on this point! > > Re-reading your previous comments, I agree that we

Re: [DISCUSS] KIP-704: Send a hint to broker if it is an unclean leader

2022-01-28 Thread José Armando García Sancio
Hi all, Jason and I discussed this offline. At a high-level I have made the following changes to the KIP. 1. IBP will be used to enable this feature and to determine which version of LeaderAndIsr and AlterPartition will be used. 2. The LeaderRecoveryState field for LeaderAndIsr and AlterPartition

Re: [VOTE] KIP-591: Add Kafka Streams config to set default state store

2022-01-28 Thread Guozhang Wang
Hi Luke, I'm in favor of using the newly proposed `#sessionStore(StoreType..)` and deprecating the existing `#persistenSessionStore` etc. Thanks, Guozhang On Tue, Jan 25, 2022 at 12:17 AM Luke Chen wrote: > Thanks Matthias! > > I agree we could deprecate the existing ones, and add the one with

Re: [VOTE] KIP-591: Add Kafka Streams config to set default state store

2022-01-28 Thread John Roesler
Hi Luke, Thanks for the KIP! I'm +1 (binding) on your KIP. Regarding this last question about chaning Materialized and/or Stores, I think it might actually be best to drop that part of the proposal. The primary benefit of your proposal is in the cases when the user doesn't want to specify the s

[GitHub] [kafka-site] showuon opened a new pull request #396: MINOR: Add CVE-2022-23302 and CVE-2022-23305 to cve-list

2022-01-28 Thread GitBox
showuon opened a new pull request #396: URL: https://github.com/apache/kafka-site/pull/396 Add log4j 1.x vulnerabilities in cve-list page: CVE-2022-23302 and CVE-2022-23305. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [kafka-site] showuon commented on pull request #396: MINOR: Add CVE-2022-23302 and CVE-2022-23305 to cve-list

2022-01-28 Thread GitBox
showuon commented on pull request #396: URL: https://github.com/apache/kafka-site/pull/396#issuecomment-1024827129 @junrao , please take a look. Thanks. -- 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