Re: [DISCUSS] KIP-1041: Drop `offsets.commit.required.acks` config in 4.0 (deprecate in 3.8)

2024-05-03 Thread Federico Valeri
Hi David, I can't think about any valid use case where changing the
default value would be useful, and your motivation is convincing.
Thanks.

On Fri, May 3, 2024 at 6:01 AM Andrew Schofield
 wrote:
>
> Hi David,
> I think this KIP is a very good idea. It would be good to get rid of this 
> cruft.
>
> Thanks,
> Andrew
>
> > On 2 May 2024, at 18:54, David Jacot  wrote:
> >
> > Hi folks,
> >
> > I have put together a very small KIP to
> > deprecate offsets.commit.required.acks in 3.8 and remove it in 4.0. See the
> > motivation for the reason.
> >
> > KIP: https://cwiki.apache.org/confluence/x/9YobEg
> >
> > Please let me know what you think.
> >
> > Best,
> > David
>


Re: [DISCUSS] KIP-1041: Drop `offsets.commit.required.acks` config in 4.0 (deprecate in 3.8)

2024-05-02 Thread Andrew Schofield
Hi David,
I think this KIP is a very good idea. It would be good to get rid of this cruft.

Thanks,
Andrew

> On 2 May 2024, at 18:54, David Jacot  wrote:
>
> Hi folks,
>
> I have put together a very small KIP to
> deprecate offsets.commit.required.acks in 3.8 and remove it in 4.0. See the
> motivation for the reason.
>
> KIP: https://cwiki.apache.org/confluence/x/9YobEg
>
> Please let me know what you think.
>
> Best,
> David



[DISCUSS] KIP-1041: Drop `offsets.commit.required.acks` config in 4.0 (deprecate in 3.8)

2024-05-02 Thread David Jacot
Hi folks,

I have put together a very small KIP to
deprecate offsets.commit.required.acks in 3.8 and remove it in 4.0. See the
motivation for the reason.

KIP: https://cwiki.apache.org/confluence/x/9YobEg

Please let me know what you think.

Best,
David