Hi Mickael,
Just took a look at your draft PR, seems like your implementation is looking to
change the way we call all consumer group APIs to leverage the new
AdminApiHandler introduced by Jason as part of KIP-664. It also seems like
someone already got started to rewrite the ListOffsets API using this new
infrastructure with this PR: https://github.com/apache/kafka/pull/10467/files
:). In any case this is very helpful and certainly changes my plans on how I
intended to implement the KIP. I can volunteer to help review your PR once
you’re done with it, since it will give me a good idea for my PR. I was also
hoping to get KIP-709 into 3.0 before the code freeze, but given that we are
now rewriting ListOffsets API, I think I’d better wait until you get your KIP
in. Are you planning on getting KIP-699 into 3.0?
Cheers,
Sanjana
On May 23, 2021, 11:35 PM -0700, Sanjana Kaundinya <skaundi...@gmail.com>,
wrote:
> Hi Mickael,
> Thank you for your response! Will take a look at the draft PR later this week.
> Cheers,
> Sanjana
> On May 21, 2021, 10:26 AM -0700, Mickael Maison <mickael.mai...@gmail.com>,
> wrote:
> > Hi Sanjana,
> >
> > Again I'm sorry for the delay.
> > I've opened a draft PR which hopefully will let you make progress on
> > KIP-709. Let me know if you have any questions.
> >
> > Thanks
> >
> > On Thu, May 20, 2021 at 10:10 PM Sanjana Kaundinya <skaundi...@gmail.com>
> > wrote:
> > >
> > > Hi Mickael,
> > >
> > > Apologies for pinging on this again, but just wondering if you got a
> > > chance to make any progress on this? Would love to take a look and help
> > > with the reviews if you have.
> > >
> > > Thanks,
> > > Sanjana
> > > On May 7, 2021, 3:33 AM -0700, Mickael Maison <mickael.mai...@gmail.com>,
> > > wrote:
> > > > Hi Sanjana,
> > > >
> > > > Sorry for the late reply.
> > > > I hope to have a draft PR next week.
> > > >
> > > > Thanks
> > > >
> > > > On Tue, May 4, 2021 at 11:11 PM Sanjana Kaundinya
> > > > <skaundi...@gmail.com> wrote:
> > > > >
> > > > > Hi Mickael,
> > > > >
> > > > > Just wanted to ping the thread to inquire if you’ve made any progress
> > > > > on the implementation. I was planning on doing the implementation for
> > > > > KIP-709 and seeing how KIP-699 is something that’s tightly coupled
> > > > > and related to KIP-709 I thought I’d ask what the status of this KIP
> > > > > was.
> > > > >
> > > > > Thanks,
> > > > > Sanjana
> > > > > On Apr 13, 2021, 8:44 AM -0700, Rajini Sivaram
> > > > > <rajinisiva...@gmail.com>, wrote:
> > > > > > Thanks Mickael!
> > > > > >
> > > > > > Regards,
> > > > > >
> > > > > > Rajini
> > > > > >
> > > > > > On Tue, Apr 13, 2021 at 4:40 PM Mickael Maison
> > > > > > <mickael.mai...@gmail.com>
> > > > > > wrote:
> > > > > >
> > > > > > > Thanks Rajini!
> > > > > > > Yes it would be good to get it into 3.0. I've got some of it
> > > > > > > implemented already, I'll try to get a PR out in the next couple
> > > > > > > of
> > > > > > > weeks.
> > > > > > >
> > > > > > > I'm closing this vote. This KIP has been accepted with 3 +1
> > > > > > > binding
> > > > > > > votes from David, Tom and Rajini.
> > > > > > >
> > > > > > > On Tue, Apr 13, 2021 at 1:49 PM Rajini Sivaram
> > > > > > > <rajinisiva...@gmail.com>
> > > > > > > wrote:
> > > > > > > >
> > > > > > > > Hi Mickael,
> > > > > > > >
> > > > > > > > +1 (binding)
> > > > > > > >
> > > > > > > > Thanks for the KIP!. Looks like this KIP has sufficient votes.
> > > > > > > > It will be
> > > > > > > > good to get this into 3.0 along with KIP-709. Will you have
> > > > > > > > time to work
> > > > > > > on
> > > > > > > > this? Please let us know if we can help with the implementation
> > > > > > > > or
> > > > > > > reviews.
> > > > > > > >
> > > > > > > > Thank you,
> > > > > > > >
> > > > > > > > Rajini
> > > > > > > >
> > > > > > > >
> > > > > > > > On Thu, Mar 18, 2021 at 4:00 PM Tom Bentley
> > > > > > > > <tbent...@redhat.com> wrote:
> > > > > > > >
> > > > > > > > > Hi Mickael,
> > > > > > > > >
> > > > > > > > > I'd like to re-cast my vote as +1 (binding) now I'm a
> > > > > > > > > committer.
> > > > > > > > >
> > > > > > > > > Thanks again,
> > > > > > > > >
> > > > > > > > > Tom
> > > > > > > > >
> > > > > > > > > On Tue, Mar 2, 2021 at 9:46 AM David Jacot
> > > > > > > > > <dja...@confluent.io>
> > > > > > > wrote:
> > > > > > > > >
> > > > > > > > > > Thanks for the KIP, Mickael. +1 (binding)
> > > > > > > > > >
> > > > > > > > > > On Mon, Mar 1, 2021 at 11:53 AM Tom Bentley
> > > > > > > > > > <tbent...@redhat.com>
> > > > > > > wrote:
> > > > > > > > > >
> > > > > > > > > > > +1 (non-binding), thanks Mickael.
> > > > > > > > > > >
> > > > > > > > > > > On Thu, Feb 25, 2021 at 6:32 PM Mickael Maison <
> > > > > > > mimai...@apache.org>
> > > > > > > > > > > wrote:
> > > > > > > > > > >
> > > > > > > > > > > > Hi,
> > > > > > > > > > > >
> > > > > > > > > > > > I'd like to start a vote on KIP-699 to support
> > > > > > > > > > > > resolving multiple
> > > > > > > > > > > > coordinators at a time:
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-699%3A+Update+FindCoordinator+to+resolve+multiple+Coordinators+at+a+time
> > > > > > > > > > > >
> > > > > > > > > > > > Thanks
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > >
