Hi, Jorge, Thanks for the KIP. Looks good to me overall. A few comments below.
1. It seems that ConsumerDescription should be MemberDescription? 2. Each offset can have an optional metadata. So, in ListGroupOffsetsResult, perhaps it's better to have KafkaFuture<Map<TopicPartition, OffsetAndMetadata>>, where OffsetAndMetadata contains an offset and a metadata of String. 3. As Jason mentioned in the discussion, it would be nice to extend this api to support general group management, instead of just the consumer group in the future. For that, it might be better for MemberDescription to have assignment of type Assignment, which consists of a list of partitions. Then, in the future, we can add other fields to Assignment. Jun On Thu, Jan 18, 2018 at 9:45 AM, Mickael Maison <[email protected]> wrote: > +1 (non binding), thanks > > On Thu, Jan 18, 2018 at 5:41 PM, Colin McCabe <[email protected]> wrote: > > +1 (non-binding) > > > > Colin > > > > > > On Thu, Jan 18, 2018, at 07:36, Ted Yu wrote: > >> +1 > >> -------- Original message --------From: Bill Bejeck <[email protected]> > >> Date: 1/18/18 6:59 AM (GMT-08:00) To: [email protected] Subject: > >> Re: [VOTE] KIP-222 - Add "describe consumer group" to KafkaAdminClient > >> Thanks for the KIP > >> > >> +1 > >> > >> Bill > >> > >> On Thu, Jan 18, 2018 at 4:24 AM, Rajini Sivaram < > [email protected]> > >> wrote: > >> > >> > +1 (binding) > >> > > >> > Thanks for the KIP, Jorge. > >> > > >> > Regards, > >> > > >> > Rajini > >> > > >> > On Wed, Jan 17, 2018 at 9:04 PM, Guozhang Wang <[email protected]> > wrote: > >> > > >> > > +1 (binding). Thanks Jorge. > >> > > > >> > > > >> > > Guozhang > >> > > > >> > > On Wed, Jan 17, 2018 at 11:29 AM, Gwen Shapira <[email protected]> > >> > wrote: > >> > > > >> > > > Hey, since there were no additional comments in the discussion, > I'd > >> > like > >> > > to > >> > > > resume the voting. > >> > > > > >> > > > +1 (binding) > >> > > > > >> > > > On Fri, Nov 17, 2017 at 9:15 AM Guozhang Wang <[email protected] > > > >> > > wrote: > >> > > > > >> > > > > Hello Jorge, > >> > > > > > >> > > > > I left some comments on the discuss thread. The wiki page itself > >> > looks > >> > > > good > >> > > > > overall. > >> > > > > > >> > > > > > >> > > > > Guozhang > >> > > > > > >> > > > > On Tue, Nov 14, 2017 at 10:02 AM, Jorge Esteban Quilcate Otoya < > >> > > > > [email protected]> wrote: > >> > > > > > >> > > > > > Added. > >> > > > > > > >> > > > > > El mar., 14 nov. 2017 a las 19:00, Ted Yu (< > [email protected]>) > >> > > > > > escribió: > >> > > > > > > >> > > > > > > Please fill in JIRA number in Status section. > >> > > > > > > > >> > > > > > > On Tue, Nov 14, 2017 at 9:57 AM, Jorge Esteban Quilcate > Otoya < > >> > > > > > > [email protected]> wrote: > >> > > > > > > > >> > > > > > > > JIRA issue title updated. > >> > > > > > > > > >> > > > > > > > El mar., 14 nov. 2017 a las 18:45, Ted Yu (< > >> > [email protected] > >> > > >) > >> > > > > > > > escribió: > >> > > > > > > > > >> > > > > > > > > Can you fill in JIRA number (KAFKA-6058 > >> > > > > > > > > <https://issues.apache.org/jira/browse/KAFKA-6058>) ? > >> > > > > > > > > > >> > > > > > > > > If one JIRA is used for the two additions, consider > updating > >> > > the > >> > > > > JIRA > >> > > > > > > > > title. > >> > > > > > > > > > >> > > > > > > > > On Tue, Nov 14, 2017 at 9:04 AM, Jorge Esteban Quilcate > >> > Otoya < > >> > > > > > > > > [email protected]> 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/pages/viewpage. > >> > > > > > > > action?pageId=74686265 > >> > > > > > > > > > > >> > > > > > > > > > Cheers, > >> > > > > > > > > > Jorge. > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > > >> > > > > > >> > > > > -- > >> > > > > -- Guozhang > >> > > > > > >> > > > > >> > > > >> > > > >> > > > >> > > -- > >> > > -- Guozhang > >> > > > >> > >
