Hey All,

Thanks everyone for the votes.

I had some offline discussion with Magnus Edenhill and he brought up a
potential problem with the change in behavior for group.(min|max).
session.timeout.ms. Unlike the java consumer, the librdkafka consumer
proactively revokes partitions after the session timeout expires locally
with no response from the coordinator. That means that the automatic
adjustment of the session timeout would not be respected by the client.
What is worse, after the consumer expires its local session timeout and
revokes partitions, it rejoins as a new member. However, the rebalance
would not be able to complete until the old member had expired, so the net
effect would be that rebalances get unexpectedly delayed by the adjusted
session timeout.

I think this means we need to give this change some more thought, so I am
removing it from the KIP. Probably we will have to bump the JoinGroup API
so that the coordinator can make the client aware of the adjusted session
timeout (and probably the heartbeat interval as well). I will look into
doing this change in a separate proposal.

Thanks,
Jason

On Thu, Apr 29, 2021 at 1:43 AM Bruno Cadonna <cado...@apache.org> wrote:

> Thank you for the KIP, Jason!
>
> +1 (binding)
>
> Best,
> Bruno
>
> On 29.04.21 10:10, Luke Chen wrote:
> > Hi Jason,
> > +1 (non-binding)
> >
> > Really need this KIP to save poor jenkins flaky tests. :)
> >
> > Luke
> >
> > On Thu, Apr 29, 2021 at 4:01 PM David Jacot <dja...@confluent.io.invalid
> >
> > wrote:
> >
> >> +1 (binding)
> >>
> >> Thanks for the KIP.
> >>
> >> On Thu, Apr 29, 2021 at 2:27 AM Bill Bejeck <bbej...@gmail.com> wrote:
> >>
> >>> Thanks for the KIP Jason, +1(binding)
> >>>
> >>> -Bill
> >>>
> >>> On Wed, Apr 28, 2021 at 7:47 PM Guozhang Wang <wangg...@gmail.com>
> >> wrote:
> >>>
> >>>> +1. Thanks Jason!
> >>>>
> >>>> On Wed, Apr 28, 2021 at 12:50 PM Gwen Shapira
> >> <g...@confluent.io.invalid
> >>>>
> >>>> wrote:
> >>>>
> >>>>> I love this improvement.
> >>>>>
> >>>>> +1 (binding)
> >>>>>
> >>>>> On Wed, Apr 28, 2021 at 10:46 AM Jason Gustafson
> >>>>> <ja...@confluent.io.invalid>
> >>>>> wrote:
> >>>>>
> >>>>>> Hi All,
> >>>>>>
> >>>>>> I'd like to start a vote on KIP-735:
> >>>>>>
> >>>>>>
> >>>>>
> >>>>
> >>>
> >>
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-735%3A+Increase+default+consumer+session+timeout
> >>>>>> .
> >>>>>> +1
> >>>>>> from myself obviously
> >>>>>>
> >>>>>> -Jason
> >>>>>>
> >>>>>
> >>>>>
> >>>>> --
> >>>>> Gwen Shapira
> >>>>> Engineering Manager | Confluent
> >>>>> 650.450.2760 | @gwenshap
> >>>>> Follow us: Twitter | blog
> >>>>>
> >>>>
> >>>>
> >>>> --
> >>>> -- Guozhang
> >>>>
> >>>
> >>
> >
>

Reply via email to