Re: KIP-327: Add describe all topics API to AdminClient

2018-07-17 Thread Colin McCabe
> > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Jul 13, 2018 at 12:38 AM Jason Gustafson > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > Hey Manikumar, > > > > > > > > > > > > > > > > > > > > As Kafka begins to scale to larger and larger numbers of > > > > > > > > topics/partitions, > > > > > > > > > > I'm a little concerned about the scalability of APIs such > > as > > > > > > > > > > this. The> > API > > > > > > > > > > looks benign, but imagine you have have a few million > > > > > > > > > > partitions. We> > > > already expose similar APIs in the > > > > producer > > > > > > and consumer, so > > > > > > > > > > probably> > not > > > > > > > > > > much additional harm to expose it in the AdminClient, but > > it > > > > > > > > > > would be> > nice > > > > > > > > > > to put a little thought into some longer term options. We > > > > should > > > > > > > > > > be> > giving > > > > > > > > > > users an efficient way to select a smaller set of the > > topics > > > > > > > > > > they are> > > > interested in. We have always discussed > > adding > > > > > > some filtering > > > > > > > > > > support> > to > > > > > > > > > > the Metadata API. Perhaps now is a good time to reconsider > > > > this? > > > > > > > > > > We now> > > > have a convention for wildcard ACLs, so > > perhaps > > > > we > > > > > > can do > > > > > > > > > > something> > > > similar. Full regex support might be ideal > > > > given > > > > > > the consumer's> > > > subscription API, but that is more > > challenging. > > > > > What > > > > > > do you > > > > > > > > > > think?> > > > > > > > > > > > > > Thanks, > > > > > > > > > > Jason > > > > > > > > > > > > > > > > > > > > On Thu, Jul 12, 2018 at 2:35 PM, Harsha > > > > > wrote:> > > > > > > > > > > > > > > > > > > > > Very useful. LGTM. > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > Harsha > > > > > > > > > > > > > > > > > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > > > > > > > > > Hi all, > > > > > > > > > > > > > > > > > > > > > > > > I have created a KIP to add describe all topics API to > > > > > > > > > > > > AdminClient> > . > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > > > > > > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > > > > > > > > > > > > > > > > > Please take a look. > > > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-17 Thread Manikumar
fson > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > Hey Manikumar, > > > > > > > > > > > > > > > > > > As Kafka begins to scale to larger and larger numbers of

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-15 Thread Colin McCabe
d larger numbers of > > > > > > topics/partitions, > > > > > > > > I'm a little concerned about the scalability of APIs such as > > > > > > > > this. The> > API > > > > > > > > looks benign, but imagine

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-15 Thread Colin McCabe
AdminClient, but it > > > > > > would be> > nice > > > > > > to put a little thought into some longer term options. We should > > > > > > be> > giving > > > > > > users an efficient way

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-14 Thread Manikumar
t; producer > > > and consumer, so > > > > > > > probably> > not > > > > > > > much additional harm to expose it in the AdminClient, but it > > > > > > > would be> > nice > > > > > > > to put a little thought into some longer term options. We > should > > > > > > > be> > giving > > > > > > > users an efficient way to select a smaller set of the topics > > > > > > > they are> > > > interested in. We have always discussed adding > > > some filtering > > > > > > > support> > to > > > > > > > the Metadata API. Perhaps now is a good time to reconsider > this? > > > > > > > We now> > > > have a convention for wildcard ACLs, so perhaps > we > > > can do > > > > > > > something> > > > similar. Full regex support might be ideal > given > > > the consumer's> > > > subscription API, but that is more challenging. > > What > > > do you > > > > > > > think?> > > > > > > > > > > Thanks, > > > > > > > Jason > > > > > > > > > > > > > > On Thu, Jul 12, 2018 at 2:35 PM, Harsha > > wrote:> > > > > > > > > > > > > > > Very useful. LGTM. > > > > > > > > > > > > > > > > Thanks, > > > > > > > > Harsha > > > > > > > > > > > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > > > > > > Hi all, > > > > > > > > > > > > > > > > > > I have created a KIP to add describe all topics API to > > > > > > > > > AdminClient> > . > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > > > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > > > > > > > > > > > Please take a look. > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > > > > > > > > > > > > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-14 Thread Ted Yu
gt; > > > > would be> > nice > > > > > > to put a little thought into some longer term options. We should > > > > > > be> > giving > > > > > > users an efficient way to select a smaller set of the topics > > > > > > they are> > > > interested in. We have always discussed adding > > some filtering > > > > > > support> > to > > > > > > the Metadata API. Perhaps now is a good time to reconsider this? > > > > > > We now> > > > have a convention for wildcard ACLs, so perhaps we > > can do > > > > > > something> > > > similar. Full regex support might be ideal given > > the consumer's> > > > subscription API, but that is more challenging. > What > > do you > > > > > > think?> > > > > > > > > > Thanks, > > > > > > Jason > > > > > > > > > > > > On Thu, Jul 12, 2018 at 2:35 PM, Harsha > wrote:> > > > > > > > > > > > > Very useful. LGTM. > > > > > > > > > > > > > > Thanks, > > > > > > > Harsha > > > > > > > > > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > > > > > Hi all, > > > > > > > > > > > > > > > > I have created a KIP to add describe all topics API to > > > > > > > > AdminClient> > . > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > > > > > > > > > Please take a look. > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > > > > > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-14 Thread Stephane Maarek
; interested in. We have always discussed adding > some filtering > > > > > support> > to > > > > > the Metadata API. Perhaps now is a good time to reconsider this? > > > > > We now> > > > have a convention for wildcard ACLs, so perhaps we > can do > > > > > something> > > > similar. Full regex support might be ideal given > the consumer's> > > > subscription API, but that is more challenging. What > do you > > > > > think?> > > > > > > > > Thanks, > > > > > Jason > > > > > > > > > > On Thu, Jul 12, 2018 at 2:35 PM, Harsha wrote:> > > > > > > > > > > Very useful. LGTM. > > > > > > > > > > > > Thanks, > > > > > > Harsha > > > > > > > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > > > > Hi all, > > > > > > > > > > > > > > I have created a KIP to add describe all topics API to > > > > > > > AdminClient> > . > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > > > > > > > Please take a look. > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-14 Thread Colin McCabe
t> > to > > > > the Metadata API. Perhaps now is a good time to reconsider this? > > > > We now> > > > have a convention for wildcard ACLs, so perhaps we can do > > > > something> > > > similar. Full regex support might be ideal given the > > > > consumer's> > > > subscription API, but that is more challenging. What > > > > do you > > > > think?> > > > > > > > Thanks, > > > > Jason > > > > > > > > On Thu, Jul 12, 2018 at 2:35 PM, Harsha wrote:> > > > > > > > > Very useful. LGTM. > > > > > > > > > > Thanks, > > > > > Harsha > > > > > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > > > Hi all, > > > > > > > > > > > > I have created a KIP to add describe all topics API to > > > > > > AdminClient> > . > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > > > > > Please take a look. > > > > > > > > > > > > Thanks, > > > > > > > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-13 Thread Ted Yu
t; > Thanks, > > > Jason > > > > > > On Thu, Jul 12, 2018 at 2:35 PM, Harsha wrote: > > > > > > > Very useful. LGTM. > > > > > > > > Thanks, > > > > Harsha > > > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > > Hi all, > > > > > > > > > > I have created a KIP to add describe all topics API to AdminClient > . > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > > > Please take a look. > > > > > > > > > > Thanks, > > > > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-13 Thread Colin McCabe
sha > > > > > > On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > > > > Hi all, > > > > > > > > I have created a KIP to add describe all topics API to AdminClient . > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > > > Please take a look. > > > > > > > > Thanks, > > > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-13 Thread Andras Beni
have created a KIP to add describe all topics API to AdminClient . > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > > > Please take a look. > > > > > > Thanks, > > >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-12 Thread Jason Gustafson
pache.org/confluence/display/KAFKA/KIP- > 327%3A+Add+describe+all+topics+API+to+AdminClient > > > > Please take a look. > > > > Thanks, >

Re: KIP-327: Add describe all topics API to AdminClient

2018-07-12 Thread Harsha
Very useful. LGTM. Thanks, Harsha On Thu, Jul 12, 2018, at 9:56 AM, Manikumar wrote: > Hi all, > > I have created a KIP to add describe all topics API to AdminClient . > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-327%3A+Add+describe+all+topics+API+to+AdminClient &

KIP-327: Add describe all topics API to AdminClient

2018-07-12 Thread Manikumar
Hi all, I have created a KIP to add describe all topics API to AdminClient . https://cwiki.apache.org/confluence/display/KAFKA/KIP-327%3A+Add+describe+all+topics+API+to+AdminClient Please take a look. Thanks,