Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-08 Thread Manikumar
> > > > > > It would also be helpful for others to give feedback on this KIP. > Rajini, > > > Gwen, Ismael? > > > > > > Thanks, > > > > > > Jun > > > > > > > > > On Sun, Feb 5, 2017 at 9:54 AM, Manikumar <m

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-05 Thread Manikumar
n authentication time. > > > 110. The diagrams in the wiki still show MD5 digest. Could you change it to > SCRAM? > > Updated the diagram. Thanks, Manikumar > > > > > > Thanks. > > Manikumar > > > > > > > > > > On Fri, De

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-08 Thread Manikumar
Agree, we can run the token expiry check thread as part of controller broker. WIll update the KIP. Thanks, Manikumar > > On Sun, Feb 5, 2017 at 9:54 AM, Manikumar <manikumar.re...@gmail.com> > wrote: > > > Hi Jun, > > > > Please see the replies inline. >

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-08 Thread Manikumar
thread for impersonation related discussion. Thanks, Manikumar On Wed, Feb 8, 2017 at 8:37 AM, Roger Hoover <roger.hoo...@gmail.com> wrote: > Hi Jun, > > How does it allow impersonation at the connection level? Looking at the > KIP, the DelegationTokenRequest does not have an &quo

Re: [DISCUSS] KIP-117: Add a public AdministrativeClient API for Kafka admin operations

2017-02-05 Thread Manikumar
+1 for ismael's suggestion. grouping of methods by the relevant resource will simply the API. In future, we will be adding delegation token related operations to admin client. I can imagine methods like adminClient.token().create(...), adminClient.token().renew(...), etc.. Thanks, Manikumar

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-01-24 Thread Manikumar
terminology tokenUID => tokenID. > 107.3 How does each broker know about changes in tokens? Does each broker > register a watcher for /tokenauth/tokens? > Yes, the changes are updated through zk notifications. > 107.4 How is token deletion handled? Does every broker delete the same

Re: [VOTE] KIP-111 Kafka should preserve the Principal generated by the PrincipalBuilder while processing the request received on socket channel, on the broker.

2017-02-22 Thread Manikumar
; > > > > > gharatmayures...@gmail.com > > > > > > wrote: > > > > > > > > > > > Hi Jun, > > > > > > > > > > > > Sure sounds good to me. > > > > > > > > > > > > Th

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-13 Thread Manikumar
Thank you all for your votes and feedback. The vote has passed with 4 binding votes(Gwen, Jun, Grant, Harsha) and 2 non-binding votes(Roger, Dong Lin). I have updated the relevant wiki pages. Thanks Manikumar On Tue, Feb 14, 2017 at 12:02 AM, Dong Lin <lindon...@gmail.com> wrote: >

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-10 Thread Manikumar
end the support to allow a user to acquire delegation tokens for other users, then we can enable CREATE/DELETE operations. Updated the KIP. Thanks, Manikumar > On Wed, Feb 8, 2017 at 1:35 AM, Manikumar <manikumar.re...@gmail.com> > wrote: > > > Hi Jun, > > > > &g

Re: [VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2017-02-10 Thread Manikumar
Yes, owners and the renewers can always describe their own tokens. Updated the KIP. On Sat, Feb 11, 2017 at 3:12 AM, Jun Rao wrote: > Hi, Mani, > > Thanks for the update. Just a minor comment below. Otherwise, +1 from me. > > > > > > > > > > 116. Could you document the ACL

Re: [VOTE] KIP-111 Kafka should preserve the Principal generated by the PrincipalBuilder while processing the request received on socket channel, on the broker.

2017-02-15 Thread Manikumar
Sorry, I am late to this discussion. PrincipalBuilder is only used for SSL Protocol. For SASL, we use "sasl.kerberos.principal.to.local.rules" config to map SASL principal names to short names. To make it consistent, Do we also need to pass the SASL full principal name to authorizer ? We may need

Re: [VOTE] KIP-111 Kafka should preserve the Principal generated by the PrincipalBuilder while processing the request received on socket channel, on the broker.

2017-02-16 Thread Manikumar
kafka-acls.sh would > > work with this change, on the ticket, we designed a detailed solution to > > make this tool generally usable with all sorts of combinations of > > Authorizers and PrincipalBuilders and give more flexibility to the end > > users. > > Without the

[DISCUSS] KIP-80: Kafka REST Server

2016-09-28 Thread Manikumar
://cwiki.apache.org/confluence/display/KAFKA/KIP- 80%3A+Kafka+Rest+Server Your comments and feedback are welcome. Thanks, Manikumar

Re: [DISCUSS] KIP-80: Kafka REST Server

2016-10-01 Thread Manikumar
ess number of distributions/builds. Thanks, Manikumar On Sat, Oct 1, 2016 at 4:24 AM, Jay Kreps <j...@confluent.io> wrote: > Hey guys, > > There's already a REST interface maintained as a separate project--it's > open source and apache licensed and actively maintained ( > https://git

Re: [kafka-clients] [VOTE] 0.10.1.0 RC3

2016-10-17 Thread Manikumar
+1 (non-binding) verified quick start and artifacts. On Sat, Oct 15, 2016 at 4:59 AM, Jason Gustafson wrote: > Hello Kafka users, developers and client-developers, > > One more RC for 0.10.1.0. We're hoping this is the final one so that we > can meet the release target date

Re: [ANNOUNCE] Apache Kafka 0.10.1.0 Released

2016-10-20 Thread Manikumar
> > > > > > > > and binary releases from > > > > > > https://www.apache.org/dyn/closer.cgi?path=/kafka/0.10.1.0/k > > > > > > afka_2.11-0.10.1.0.tgz > > > > > > https://www.apache.org/dyn/closer.cgi?path=/kafka/0.10.1.0/k

Re: [DISCUSS] KIP-80: Kafka REST Server

2016-11-04 Thread Manikumar
for raising valid queries and helping me to improve the KIP details. I will discuss some of your notes (new consumer API, HTTP/2, multi-REST-proxy etc) at kafka-rest project issues page . Thanks you all for participating in this discussion. Thanks, Manikumar On Wed, Oct 26, 2016 at 11:56 AM

Re: Reg: ACLS

2016-12-11 Thread Manikumar
The default principal for Plaintext transport is "ANONYMOUS". On Sun, Dec 11, 2016 at 12:58 AM, BigData dev wrote: > Hi, > > bin/kafka-acls.sh --topic kafka-testtopic --add -allow-host 9.30.15.19 > --operation Write --authorizer-properties >

Re: [DISCUSS] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-13 Thread Manikumar
en for a specified amount of time? In > current version of KIP, renew request does not take time as a param, not > sure what is expiry time set to after renewal. > > Yes, we need to specify renew period. I'll update the KIP. Thanks, Mankumar > > On Mon, Dec 12, 2016 at 9:08 AM

Re: [DISCUSS] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-14 Thread Manikumar
? > Kafka Connect, NiFi and friends can probably use this to share clients > between multiple jobs, tasks, etc. > > What do you think? > > Gwen > > On Tue, Dec 13, 2016 at 12:43 AM, Manikumar <manikumar.re...@gmail.com> > wrote: > > > Ashish, > > >

Re: [DISCUSS] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-15 Thread Manikumar
give any suggestions/concerns on this KIP. Thanks, > > On Thu, Dec 15, 2016 at 9:04 AM, Manikumar <manikumar.re...@gmail.com> > wrote: > > > @Gwen, @Rajini, > > > > As mentioned in the KIP, main motivation for this KIP is to reduce load > on > > Kerberos >

Re: [DISCUSS] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-15 Thread Manikumar
for authentication and tokenID for retrial of tokenHmac at server side. Does above sound OK? Thanks, Manikumar On Wed, Dec 14, 2016 at 10:33 PM, Harsha Chintalapani <ka...@harsha.io> wrote: > @Gwen @Mani Not sure why we want to authenticate at every request. Even if > the to

Re: [DISCUSS] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-12 Thread Manikumar
+Delegation+token+support+for+Kafka Jun, Ashish, Gwen, Pl review the updated KIP. Thanks Manikumar On Thu, Sep 29, 2016 at 9:56 PM, Ashish Singh <asi...@cloudera.com> wrote: > Harsha/ Gwen, > > How do we proceed here? I am willing to help out with here. > > On Fri, Sep 23, 2

[VOTE] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-23 Thread Manikumar
Hi, I would like to initiate the vote on KIP-48: https://cwiki.apache.org/confluence/display/KAFKA/KIP-48+ Delegation+token+support+for+Kafka Thanks, Manikumar

Re: [DISCUSS] KIP-48 Support for delegation tokens as an authentication mechanism

2016-12-20 Thread Manikumar
Hi Devs, If there are no more comments, I will start vote on this KIP later this week. Thanks On Fri, Dec 16, 2016 at 12:28 PM, Manikumar <manikumar.re...@gmail.com> wrote: > Hi, > > >> Can you add a sample Jaas configuration using delegation tokens to the >> KIP?

Re: [VOTE] 0.10.2.1 RC1

2017-04-17 Thread Manikumar
+1 (non-binding) Verified artifacts and quick start On Mon, Apr 17, 2017 at 8:05 PM, Mathieu Fenniak < mathieu.fenn...@replicon.com> wrote: > +1 (non-binding) > > Upgraded KS & KC applications to 0.10.2.1 RC1, successfully ran > application-level acceptance tests. > > Mathieu > > > On Wed, Apr

Re: Doubts regarding KafkaProducer implemetation

2017-03-13 Thread Manikumar
Hi, java producer waits for the metadata of the given topic to be available. We will get TimeoutException if we didn't get metadata within max.block.ms. As observed by you, this behavior is different from old producer. "retries" config is not applicable to metadata fetch. We can adjust

Re: [VOTE] KIP-163: Lower the Minimum Required ACL Permission of OffsetFetch

2017-08-11 Thread Manikumar
+1 (non-binding) On Fri, Aug 11, 2017 at 5:31 AM, Jun Rao wrote: > Hi, Vahid, > > Thanks for the KIP. +1 from me. > > Jun > > On Wed, Aug 2, 2017 at 2:40 PM, Vahid S Hashemian < > vahidhashem...@us.ibm.com > > wrote: > > > Hi all, > > > > Thanks to everyone who participated

Re: Usage of SSL_ENDPOINT_IDENTIFICATION_ALGORITHM_CONFIG

2017-08-10 Thread Manikumar
check here: https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/security/ssl/SslFactory.java#L84 On Thu, Aug 10, 2017 at 3:49 AM, M. Manna wrote: > Hello, > > I have been trying to find the usage of this property within Kafka source. >

Re: [VOTE] KIP-175: Additional '--describe' views for ConsumerGroupCommand

2017-08-14 Thread Manikumar
+1 (non-binding) On Fri, Aug 11, 2017 at 8:09 PM, Mickael Maison wrote: > +1 non-binding, thanks Vahid > > On Wed, Aug 9, 2017 at 9:31 PM, Jason Gustafson > wrote: > > Thanks for the KIP. +1 > > > > On Thu, Jul 27, 2017 at 2:04 PM, Vahid S

Re: [DISCUSS] KIP-188 - Add new metrics to support health checks

2017-08-18 Thread Manikumar
I agree it will be good if we can add "commit id/version" as an attribute value. It will be easy to parse. But as of now, KafkaMetric supports only numerical values. On Fri, Aug 18, 2017 at 5:49 AM, Ismael Juma wrote: > Hi Rajini, > > About the gauges, I was thinking that

Re: [DISCUSS] KIP-185: Make exactly once in order delivery per partition the default producer setting

2017-08-18 Thread Manikumar
I agree with Jason/Apruva comments. we can wait some more time and enable it in 1.1.0 release. In general, we are in agreement with the change. On Fri, Aug 18, 2017 at 10:32 AM, Apurva Mehta wrote: > Hi Jason, > > You make some good points. > > First, I agree that we could

Re: [DISCUSS] KIP-185: Make exactly once in order delivery per partition the default producer setting

2017-08-18 Thread Manikumar
> > 3. The message format requirement is a good point. This should be mentioned > in the compatibility section. Users who are still using the old message > format will get an error after the upgrade, right? > so this change will break client backward compatibility while connecting to 0.10.X

Re: MetricsReporter Does Not Receive Auto-generated Client Id

2017-09-14 Thread Manikumar
Hi, I think there is no specific reason. we can pass the clientId to MetricsReporter class's configure() method. Can you pls raise the JIRA? On Fri, Sep 15, 2017 at 7:44 AM, Kevin Lu wrote: > Hi, > > Is there a particular reason why the MetricsReporter class's

KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-09-18 Thread Manikumar
. Manikumar

Re: KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-09-19 Thread Manikumar
ent...@gmail.com> wrote: > What locale is used for the case conversion, the JVM default one or a > specific one? > > On 18 Sep 2017 5:31 pm, "Manikumar" <manikumar.re...@gmail.com> wrote: > > > Hi all, > > > > I've created a small KIP to extend the sasl

Re: [DISCUSS] KIP-186: Increase offsets retention default to 7 days

2017-10-06 Thread Manikumar
looks like VOTE thread is *NOT* started for this KIP. On Fri, Oct 6, 2017 at 11:23 PM, Manikumar <manikumar.re...@gmail.com> wrote: > looks like VOTE thread is started for this KIP. > > > On Wed, Aug 16, 2017 at 5:39 PM, Stevo Slavić <ssla...@gmail.com> wrote: >

Re: [DISCUSS] KIP-186: Increase offsets retention default to 7 days

2017-10-06 Thread Manikumar
looks like VOTE thread is started for this KIP. On Wed, Aug 16, 2017 at 5:39 PM, Stevo Slavić wrote: > +1 for making consistent default log and offsets retention time. > I like Stephane's suggestion too, log retention override should override > offset retention too if not

Re: [VOTE] KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-10-09 Thread Manikumar
I'm bumping this up to get some attention :) On Wed, Sep 27, 2017 at 8:46 PM, Tom Bentley <t.j.bent...@gmail.com> wrote: > +1 (nonbinding) > > On 27 September 2017 at 16:10, Manikumar <manikumar.re...@gmail.com> > wrote: > > > Hi All, > > > > I

Re: [VOTE] KIP-189 - Improve principal builder interface and add support for SASL

2017-08-30 Thread Manikumar
+1 (non-binding). On Wed, Aug 30, 2017 at 9:29 PM, Mayuresh Gharat wrote: > +1 (non-binding) > > Thanks, > > Mayuresh > > On Wed, Aug 30, 2017 at 8:51 AM, Jason Gustafson > wrote: > > > I'd like to open the vote for KIP-189: > >

Re: [VOTE] 0.11.0.1 RC0

2017-09-12 Thread Manikumar
+1 (non-binding) , verified artifacts, quick start using binary and ran tests on source artifact. Thanks for running the release. On Tue, Sep 12, 2017 at 4:30 PM, Ismael Juma wrote: > Hi Damian, > > Thanks for managing the release. This may just be the first ever RC0 to be >

Re: KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-09-25 Thread Manikumar
Hi all, If there are no further concerns, I will start a VOTE thread for this minor KIP. Thanks On Tue, Sep 19, 2017 at 3:14 PM, Manikumar <manikumar.re...@gmail.com> wrote: > Thanks for the reviews. > > @Ted > Updated the KIP. > > @Tom > I think we can in

[VOTE] KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-09-27 Thread Manikumar
Hi All, I'd like to start the vote on KIP-203. Details are here: https://cwiki.apache.org/confluence/display/KAFKA/KIP-203%3A+Add+toLowerCase+support+to+sasl.kerberos.principal.to.local+rule Thanks,

Re: [VOTE] KIP-152 - Improve diagnostics for SASL authentication failures

2017-08-25 Thread Manikumar
+1 (non-binding) On Fri, Aug 25, 2017 at 12:37 AM, Vahid S Hashemian < vahidhashem...@us.ibm.com> wrote: > +1 > > Thanks Rajini. > > --Vahid > > > > From: Edoardo Comar > To: dev@kafka.apache.org > Date: 08/24/2017 10:55 AM > Subject:Re: [VOTE] KIP-152 -

Re: [VOTE] KIP-187 - Add cumulative count metric for all Kafka rate metrics

2017-08-29 Thread Manikumar
+1 (non-binding) On Tue, Aug 29, 2017 at 2:43 PM, Stevo Slavić wrote: > +1 (non-binding) > > On Tue, Aug 29, 2017 at 11:09 AM, Ismael Juma wrote: > > > Thanks for the KIP, +1 (binding) from me. > > > > Ismael > > > > On Thu, Aug 24, 2017 at 6:48 PM, Rajini

Re: [VOTE] KIP-191: KafkaConsumer.subscribe() overload that takes just Pattern

2017-08-31 Thread Manikumar
+1 (non-binding) On Thu, Aug 31, 2017 at 1:53 PM, Ismael Juma wrote: > Thanks for the KIP, +1 (binding). > > Ismael > > On 31 Aug 2017 8:38 am, "Attila Kreiner" wrote: > > Hi All, > > Thx for the comments, I pretty much see a consensus here. So I'd like to

Re: [DISCUSS] KIP-188 - Add new metrics to support health checks

2017-08-31 Thread Manikumar
Hi Rajini, Yes, It provides the failure rate for all errors. I got confused with old metric names. Thank you for the clarification. Thanks On Thu, Aug 31, 2017 at 6:34 PM, Rajini Sivaram <rajinisiva...@gmail.com> wrote: > Hi Manikumar, > > We currently have topi

Re: [DISCUSS] KIP-188 - Add new metrics to support health checks

2017-08-31 Thread Manikumar
> > > > > > > > > > Jun > > > > > > > > > > > > On Fri, Aug 18, 2017 at 1:55 AM, Rajini Sivaram < > > > > rajinisiva...@gmail.com > > > > > > > > > > > > wrote: > > > >

Re: [VOTE] KIP-188 - Add new metrics to support health checks

2017-09-04 Thread Manikumar
+1 (non-binding) On Mon, Sep 4, 2017 at 6:39 PM, Rajini Sivaram wrote: > All the suggestions on the discuss thread have been incorporated into the > KIP. Please let me know if you have any more concerns or else can we > proceed with voting for this KIP? > > Thank you, >

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-18 Thread Manikumar
+ (non-binding) Thanks, Manikumar On Tue, Oct 17, 2017 at 7:42 AM, Dong Lin <lindon...@gmail.com> wrote: > Thanks for the KIP. +1 (non-binding) > > On Wed, Oct 11, 2017 at 2:27 AM, Ted Yu <yuzhih...@gmail.com> wrote: > > > +1 > > > > On Mon, Oct 2

Re: [VOTE] KIP-223 - Add per-topic min lead and per-partition lead metrics to KafkaConsumer

2017-11-30 Thread Manikumar
+1 (non-binding) On Thu, Nov 30, 2017 at 8:35 AM, Hu Xi wrote: > Hi all, > > As I didn't see any further discussion around this KIP, I'd like to start > voting. > > KIP documentation: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- >

Re: [VOTE] KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-11-03 Thread Manikumar
Bump up. waiting for few more binding votes. On Wed, Oct 18, 2017 at 6:57 PM, Rajini Sivaram <rajinisiva...@gmail.com> wrote: > +1 (binding) > > On Mon, Oct 9, 2017 at 5:32 PM, Manikumar <manikumar.re...@gmail.com> > wrote: > > > I'm bumping this up to get some

Re: Use self contained tokens instead of ACL

2017-11-02 Thread Manikumar
Hi, Yes, the token details and scram credentials are stored in Zookeeper and clients connects using tokenId and hmac for scram authentication. We plan to implement token storage as pluggable interface. Thanks, Manikumar On Thu, Nov 2, 2017 at 2:09 PM, Postmann, P. (Peter) < peter.po

Re: [VOTE] KIP-203: Add toLowerCase support to sasl.kerberos.principal.to.local rule

2017-12-04 Thread Manikumar
This vote has passed with 3 binding votes (Rajini, Jason, Guozhang) and one non-binding vote (Tom). Thanks you all for your participation. Thanks, Manikumar On Sat, Dec 2, 2017 at 4:07 AM, Guozhang Wang <wangg...@gmail.com> wrote: > Made a pass over the KIP. +1. > > Thanks!

Re: [VOTE] Allowing write access to GitHub repositories (aka GitBox)

2017-12-12 Thread Manikumar
+1 On Tue, Dec 12, 2017 at 9:49 PM, Rajini Sivaram wrote: > +1 > > Thanks, Ismael! > > On Tue, Dec 12, 2017 at 4:18 PM, Damian Guy wrote: > > > +1 > > > > On Tue, 12 Dec 2017 at 15:47 Ismael Juma wrote: > > > > > Hi all, > > >

Re: [VOTE] KIP-214: Add zookeeper.max.in.flight.requests config to the broker

2017-10-29 Thread Manikumar
+1 (non-binding) On Fri, Oct 27, 2017 at 10:46 PM, Ted Yu wrote: > +1 > > On Fri, Oct 27, 2017 at 10:15 AM, Onur Karaman < > onurkaraman.apa...@gmail.com > > wrote: > > > I'd like to start the vote for KIP-214: Add > > zookeeper.max.in.flight.requests config to the broker >

Re: Use self contained tokens instead of ACL

2017-10-30 Thread Manikumar
lie...@opencore.com.invalid> wrote: > Hi Manikumar, > > I've looked over the KIP and had a quick look at the code in the PR as > well. In principle I think this would help Peter along depending on how > plugable some of the components are. Since Peter wants to generate Tokens > not in

Re: [VOTE] 1.0.0 RC4

2017-10-31 Thread Manikumar
+1 (non-binding). Verified quickstart, ran producer/consumer perf scripts, streams quickstart ran tests on src distribution. On Tue, Oct 31, 2017 at 8:42 PM, Ismael Juma wrote: > +1 (binding) from me. Tested the quickstart with the source and binary > (Scala 2.12) artifacts,

Re: [VOTE] KIP-266: Add TimeoutException for KafkaConsumer#position

2018-05-10 Thread Manikumar
+1 (non-binding). Thanks. On Thu, May 10, 2018 at 2:33 PM, Mickael Maison wrote: > +1 (non binding) > Thanks > > On Thu, May 10, 2018 at 9:39 AM, Rajini Sivaram > wrote: > > Hi Richard, Thanks for the KIP. > > > > +1 (binding) > > > > Regards,

Re: [VOTE] KIP-294 - Enable TLS hostname verification by default

2018-05-10 Thread Manikumar
+1 (non-binding) Thanks. On Wed, May 9, 2018 at 10:09 PM, Mickael Maison wrote: > +1, thanks for the KIP! > > On Wed, May 9, 2018 at 4:41 PM, Ted Yu wrote: > > +1 > > > > On Wed, May 9, 2018 at 8:28 AM, Rajini Sivaram >

Re: [VOTE] KIP-278: Add version option to Kafka's commands

2018-05-11 Thread Manikumar
+1 (non-binding) Thanks for the KIP. On Fri, May 11, 2018 at 12:56 PM, zhenya Sun wrote: > +1 building > > 在 2018年5月11日,上午9:51,Ted Yu 写道: > > > > +1 > > > > On Thu, May 10, 2018 at 6:42 PM, Sasaki Toru > > wrote: > > > >> Hi

Re: [VOTE] KIP-255: OAuth Authentication via SASL/OAUTHBEARER

2018-05-16 Thread Manikumar
+1 (non-binding) Thanks for the detailed KIP. On Wed, May 16, 2018 at 5:59 PM, Mickael Maison wrote: > Thanks for the KIP, > +1 (non binding) > > On Wed, May 16, 2018 at 2:51 AM, Ron Dagostino wrote: > > Hi Jun. I think you are getting at the fact

[DISCUSS] KIP-322: Return new error code for DeleteTopics API when topic deletion disabled.

2018-06-26 Thread Manikumar
Hi all, I have created a minor KIP to return new error code for DeleteTopics API when topic deletion disabled. https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=87295558 Please take a look. Thanks,

Re: [VOTE] KIP-324: Add method to get metrics() in AdminClient

2018-06-27 Thread Manikumar
+1 (non-binding) Thanks. On Wed, Jun 27, 2018 at 10:15 PM Matthias J. Sax wrote: > +1 (binding) > > On 6/26/18 2:33 PM, Guozhang Wang wrote: > > +1. Thanks. > > > > On Tue, Jun 26, 2018 at 2:31 PM, Yishun Guan wrote: > > > >> Hi All, > >> > >> I am starting a vote on this KIP: > >> > >>

Re: [kafka-clients] Re: [VOTE] 1.1.1 RC1

2018-06-25 Thread Manikumar
+1 (non-binding) Ran tests, Verified quick start, producer/consumer perf tests On Sat, Jun 23, 2018 at 8:11 AM Dong Lin wrote: > Thank you for testing and voting the release! > > I noticed that the date for 1.1.1-rc1 is wrong. Please kindly test and > vote by Tuesday, June 26, 12 pm PT. > >

Re: [VOTE] 1.0.2 RC0

2018-06-25 Thread Manikumar
+1 (non-binding) Verified tests, quick start, producer/consumer perf tests. On Sat, Jun 23, 2018 at 2:25 AM Ted Yu wrote: > +1 > > Ran test suite. > > Checked signatures. > > On Fri, Jun 22, 2018 at 11:42 AM, Vahid S Hashemian < > vahidhashem...@us.ibm.com> wrote: > > > +1 (non-binding) > > > >

Re: [DISCUSS] KIP-322: Return new error code for DeleteTopics API when topic deletion disabled.

2018-06-27 Thread Manikumar
Hi, Thanks for the review. > nit: you are going to fill in an actual value below in your PR, right ? > private static final long serialVersionUID = 1L; > > We normally set the serialVersionUID = 1L in Kafka exception classes. In Motivation, please mention the IllegalStateException

Re: [kafka-clients] [VOTE] 2.0.0 RC1

2018-06-30 Thread Manikumar
looks like maven artifacts are not updated in the staging repo. They are still at old timestamp. https://repository.apache.org/content/groups/staging/org/apache/kafka/kafka_2.11/2.0.0/ On Sat, Jun 30, 2018 at 12:06 AM Rajini Sivaram wrote: > Hello Kafka users, developers and client-developers,

Re: [kafka-clients] Re: [VOTE] 0.11.0.3 RC0

2018-07-01 Thread Manikumar
+1 (non-binding) Verified quick start, ran producer/consumer perf tests, Thanks On Sun, Jul 1, 2018 at 8:29 PM Ismael Juma wrote: > +1 (binding) > > Verified signature of source artifact, ran tests and verified quickstart > on source artifact with Java 7, verified quickstart on binary

Re: [kafka-clients] Re: [VOTE] 1.0.2 RC1

2018-07-01 Thread Manikumar
+1 (non-binding) Verified tests, quick start, producer/consumer perf tests. On Sun, Jul 1, 2018 at 11:27 PM Ismael Juma wrote: > +1 (binding) > > Verified signature of source artifact, ran tests and verified quickstart > on source artifact with Java 7, verified quickstart on binary artifact

Re: [DISCUSS] KIP-322: Return new error code for DeleteTopics API when topic deletion disabled.

2018-07-02 Thread Manikumar
Since this is a minor KIP, If there are no concerns I will start voting thread. On Wed, Jun 27, 2018 at 2:24 PM Ted Yu wrote: > bq. set the serialVersionUID = 1L in Kafka exception classes > > Indeed. Should have checked earlier. > > On Wed, Jun 27, 2018 at 1:38 AM, Man

Re: [DISCUSS] KIP-278: Add version option to Kafka's commands

2018-05-01 Thread Manikumar
I assume the intent of the KIP to find out the Kafka broker version. In this case, maybe we should expose version using a Kafka request. This will help the remote scripts/tools to query the Kafka version. scripts (kafka-topics.sh, kafka-configs.sh, etc..) may run from remote machines and may

Re: [VOTE] KIP-277 - Fine Grained ACL for CreateTopics API

2018-04-30 Thread Manikumar
+1 (non-binding) Thanks On Thu, Apr 26, 2018 at 9:59 PM, Colin McCabe wrote: > +1 (non-binding) > > best, > Colin > > > On Wed, Apr 25, 2018, at 02:45, Edoardo Comar wrote: > > Hi, > > > > The discuss thread on KIP-277 ( > >

Re: [kafka-clients] [VOTE] 1.0.0 RC2

2017-10-19 Thread Manikumar
just a minor observation: Kafka binary contains two versions of javasist library (javassist-3.20.0-GA.jar, javassist-3.21.0-GA.jar). This dependency is coming from connect project. On Thu, Oct 19, 2017 at 2:16 AM, Guozhang Wang wrote: > Thanks for pointing out, Jun, Ismael.

Re: Use self contained tokens instead of ACL

2017-10-27 Thread Manikumar
Hi, We have a accepted KIP for adding delegation token support for Kafka. https://cwiki.apache.org/confluence/display/KAFKA/KIP-48+Delegation+token+support+for+Kafka currently the PR is under review process. Maybe this can used be as starting point for your requirement.

Re: [VOTE] KIP-207:The Offsets which ListOffsetsResponse returns should monotonically increase even during a partition leader change

2017-10-26 Thread Manikumar
Thanks for the KIP. +1 (non-binding) On Thu, Oct 26, 2017 at 5:58 AM, Jason Gustafson wrote: > +1. Thanks for the KIP. > > On Mon, Oct 23, 2017 at 11:30 AM, Colin McCabe wrote: > > > On Mon, Oct 23, 2017, at 10:29, Jason Gustafson wrote: > > > Thanks

[DISCUSS] KIP-249: Add Delegation Token Operations to Kafka Admin Client

2018-01-16 Thread Manikumar
Feedback and suggestions are welcome. Thanks Manikumar

Re: [DISCUSS] KIP-249: Add Delegation Token Operations to Kafka Admin Client

2018-01-19 Thread Manikumar
Hi all, We want to include this KIP in the upcoming 1.1.0 release. Please let me know if there are any other comments. If there are no more comments, I'd like to start vote on this KIP. Thanks, On Wed, Jan 17, 2018 at 8:20 AM, Manikumar <manikumar.re...@gmail.com> wrote: > Hi, Jun, &

[VOTE] KIP-249: Add Delegation Token Operations to Kafka Admin Client

2018-01-21 Thread Manikumar
Hi All, I would like to start a vote on KIP-249 which would add delegation token operations to Java Admin Client. We have merged DelegationToken API PR recently. We want to include admin client changes in the upcoming release. This will make the feature complete.

Re: [DISCUSS] KIP-249: Add Delegation Token Operations to Kafka Admin Client

2018-01-16 Thread Manikumar
> DelegationToken. > So, it seems it's more consistent to pass in hmac as byte[] too. > 2. Does describeDelegationToken() return all tokens? > 3. As Ted mentioned, it would be useful to include the exceptions that can > be thrown in the new apis. > > Jun > > On Tue

Re: [DISCUSS] KIP-249: Add Delegation Token Operations to Kafka Admin Client

2018-01-16 Thread Manikumar
it seems it's more consistent to pass in hmac as byte[] too. > 2. Does describeDelegationToken() return all tokens? > 3. As Ted mentioned, it would be useful to include the exceptions that can > be thrown in the new apis. > > Jun > > On Tue, Jan 16, 2018 at 10:03 AM, Manikuma

Re: Kafka Log deletion Problem

2018-02-02 Thread Manikumar
looks like log segment is not rotated. You can send more data or adjust log.roll.ms/log.segment.bytes configs to rotate segments. On Fri, Feb 2, 2018 at 7:08 PM, SenthilKumar K wrote: > Hello Experts , We have a Kafka Setup running for our analytics pipeline > ...Below

Re: [DISCUSS] KIP-252: Extend ACLs to allow filtering based on ip ranges and subnets

2018-02-01 Thread Manikumar
, Feb 1, 2018 at 5:56 PM, Sönke Liebau < soenke.lie...@opencore.com.invalid> wrote: > Hi Manikumar, > > the current proposal indeed leaves out IPv6 addresses, as I was unsure > whether Kafka fully supports that yet to be honest. But it would be > fairly easy to add these to the p

Re: [DISCUSS] KIP-252: Extend ACLs to allow filtering based on ip ranges and subnets

2018-02-01 Thread Manikumar
Hi, 1. Do we support IPv6 CIDR/ranges? 2. KAFKA-5713 is mentioned in Related JIRAs section. But there is no mention of wildcard support in the KIP. Thanks, On Thu, Feb 1, 2018 at 4:05 AM, Sönke Liebau < soenke.lie...@opencore.com.invalid> wrote: > Hey everybody, > > following a brief inital

Re: [VOTE] KIP-206: Add support for UUID serialization and deserialization

2018-01-30 Thread Manikumar
+1 (non-binding) On Tue, Jan 30, 2018 at 11:50 AM, Ewen Cheslack-Postava wrote: > +1 (binding) > > On Fri, Jan 26, 2018 at 9:16 AM, Colin McCabe wrote: > > > +1 (non-binding) > > > > > > > > On Fri, Jan 26, 2018, at 08:29, Ted Yu wrote: > > > +1 > > > > >

Re: [VOTE] 1.0.1 RC2

2018-02-26 Thread Manikumar
+1 (non-binding) Built src and ran tests Ran core quick start On Sat, Feb 24, 2018 at 8:44 PM, Jakub Scholz wrote: > +1 (non-binding) ... I used the Scala 2.12 binaries and run my tests with > producers / consumers. > > On Thu, Feb 22, 2018 at 1:06 AM, Ewen Cheslack-Postava

Re: [VOTE] KIP-237: More Controller Health Metrics

2018-01-03 Thread Manikumar
+1 (non-binding) Thanks On Wed, Jan 3, 2018 at 10:48 PM, Jun Rao wrote: > Hi, Dong, > > Thanks for the KIP. +1 > > Jun > > On Mon, Dec 18, 2017 at 2:03 PM, Dong Lin wrote: > > > Hi all, > > > > Since there are no more outstanding comments, I would like

Re: [VOTE] 1.1.1 RC0

2018-06-21 Thread Manikumar
+1 (non-binding) Ran test, Verified quick start, producer/consumer perf tests On Thu, Jun 21, 2018 at 11:58 AM zhenya Sun wrote: > +1 non-binding > > > 在 2018年6月21日,下午2:18,Andras Beni 写道: > > > > +1 (non-binding) > > > > Built .tar.gz, created a cluster from it and ran a basic end-to-end

[DISCUSS] KIP-308: Support dynamic update of max.connections.per.ip/max.connections.per.ip.overrides configs

2018-06-22 Thread Manikumar
Hi all, I have created a KIP to add support for dynamic update of max.connections.per.ip/max.connections.per.ip.overrides configs *https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=85474993 * Any feedback

Re: [DISCUSS] KIP-332: Update AclCommand to use AdminClient API

2018-08-01 Thread Manikumar
Hi all, If there are no concerns, I will start the voting process soon. Thanks On Tue, Jul 31, 2018 at 9:08 AM Manikumar wrote: > Hi Colin, > > Yes, "--authorizer-properties" option is not required with > "--bootstrap-server" option. Updated the KIP. > &

[VOTE] KIP-332: Update AclCommand to use AdminClient API

2018-08-02 Thread Manikumar
/msg90049.html Thanks, Manikumar

Re: [DISCUSS] KIP-332: Update AclCommand to use AdminClient API

2018-08-03 Thread Manikumar
Updated the KIP. added a note to KIP-340 discussion thread. On Fri, Aug 3, 2018 at 2:52 PM Rajini Sivaram wrote: > Thanks Manikumar. Can you add a note to the KIP-340 discussion thread? > > On Fri, Aug 3, 2018 at 10:04 AM, Manikumar > wrote: > > > Hi Rajini, > >

Re: [DISCUSS] KIP-340: Allow kafka-reassign-partitions.sh and kafka-log-dirs.sh to take admin client property file

2018-08-03 Thread Manikumar
Hi Dong, In KIP-332 discussion, It was agreed to use "--command-config" option name for passing config property file. We can also use same name in here, to make it consistent across all tools. Thanks, On Thu, Jul 12, 2018 at 9:20 AM Dong Lin wrote: > wiki page is currently read-only and it

Re: [DISCUSS] KIP-332: Update AclCommand to use AdminClient API

2018-08-03 Thread Manikumar
e are no concerns, I will change the option to "--command-config". Since KIP-340 PR is not yet merged, we can also change there. Thanks, On Fri, Aug 3, 2018 at 1:57 PM Rajini Sivaram wrote: > Hi Manikumar, > > We have some tools already (ConfigCommand, ConsumerGroupCommand, > De

Re: [ANNOUNCE] Apache Kafka 2.0.0 Released

2018-07-30 Thread Manikumar
y, Dong Lin, Edoardo Comar, emmanuel Harel, > > Eugene Sevastyanov, Ewen Cheslack-Postava, Fedor Bobin, fedosov-alexander, > > Filipe Agapito, Florian Hussonnois, fredfp, Gilles Degols, gitlw, Gitomain, > > Guangxian, Gunju Ko, Gunnar Morling, Guozhang Wang, hmcl, huxi, huxihx, >

Re: [DISCUSS] KIP-332: Update AclCommand to use AdminClient API

2018-07-30 Thread Manikumar
Bumping this up! On Mon, Jul 23, 2018 at 8:00 PM Manikumar wrote: > Hi all, > > I have created a KIP to use AdminClient API in AclCommand (kafka-acls.sh) > > > *https://cwiki.apache.org/confluence/display/KAFKA/KIP-332%3A+Update+AclCommand+to+use+AdminClient+API* > > &

Re: [DISCUSS] KIP-332: Update AclCommand to use AdminClient API

2018-07-30 Thread Manikumar
Hi Colin, Yes, "--authorizer-properties" option is not required with "--bootstrap-server" option. Updated the KIP. Thanks, On Tue, Jul 31, 2018 at 1:30 AM Ted Yu wrote: > Look good to me. > > On Mon, Jul 23, 2018 at 7:30 AM Manikumar > wrote: > >

Re: [VOTE] KIP-289: Improve the default group id behavior in KafkaConsumer

2018-08-08 Thread Manikumar
+1 (non-binding). Thanks for the KIP. On Wed, Aug 8, 2018 at 7:36 PM Ted Yu wrote: > +1 > > On Wed, Aug 8, 2018 at 4:09 AM Mickael Maison > wrote: > > > +1 (non-binding) > > Thanks Vahid > > On Wed, Aug 8, 2018 at 11:26 AM Kamal Chandraprakash > > wrote: > > > > > > +1 (non-binding) > > > >

Re: Java 11 support

2018-08-16 Thread Manikumar
check this: https://issues.apache.org/jira/browse/KAFKA-7264 On Thu, Aug 16, 2018 at 2:40 PM Manoj Jain wrote: > Hi All, > > I have just subscribed to this group. I don't know what is format of > questions to be submitted yet but in hurry thought of getting more details > on kafka java

[DISCUSS] KIP-357: Add support to list ACLs per principal

2018-08-16 Thread Manikumar
Hi all, I have created a minor KIP to add support to list ACLs per principal using AclCommand (kafka-acls.sh) https://cwiki.apache.org/confluence/display/KAFKA/KIP-357%3A++Add+support+to+list+ACLs+per+principal Please take a look. Thanks,

  1   2   3   4   5   6   7   8   9   10   >