Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-09 Thread Tao Jiuming
+1 Yunze Xu 于2024年11月6日 周三20:46写道: > Hi all, > > I wrote a PIP (https://github.com/apache/pulsar/pull/23567) to enable > batch index ACK by default. > > From my experiences, the default ACK semantics are really confusing > and it's hard to explain the implementation details to users. Even if > it

Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-08 Thread Enrico Olivelli
+1 Thanks Enrico Il giorno ven 8 nov 2024 alle ore 12:40 Zike Yang ha scritto: > +1 > > Thanks, > Zike Yang > > On Fri, Nov 8, 2024 at 2:51 PM Zixuan Liu wrote: > > > > +1 > > > > Thanks, > > Zixuan > > > > Yunze Xu 于2024年11月6日周三 20:45写道: > > > > > Hi all, > > > > > > I wrote a PIP (https://

Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-08 Thread Zike Yang
+1 Thanks, Zike Yang On Fri, Nov 8, 2024 at 2:51 PM Zixuan Liu wrote: > > +1 > > Thanks, > Zixuan > > Yunze Xu 于2024年11月6日周三 20:45写道: > > > Hi all, > > > > I wrote a PIP (https://github.com/apache/pulsar/pull/23567) to enable > > batch index ACK by default. > > > > From my experiences, the defa

Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-07 Thread Zixuan Liu
+1 Thanks, Zixuan Yunze Xu 于2024年11月6日周三 20:45写道: > Hi all, > > I wrote a PIP (https://github.com/apache/pulsar/pull/23567) to enable > batch index ACK by default. > > From my experiences, the default ACK semantics are really confusing > and it's hard to explain the implementation details to us

Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-07 Thread WenZhi Feng
+1. On 2024/11/06 12:44:42 Yunze Xu wrote: > Hi all, > > I wrote a PIP (https://github.com/apache/pulsar/pull/23567) to enable > batch index ACK by default. > > From my experiences, the default ACK semantics are really confusing > and it's hard to explain the implementation details to users. Eve

Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-07 Thread Baodi Shi
+1 Not enabling batchIndexAck will cause confusion for users. In the future, we could even remove the client-maintained bitset to keep the client logic simple. Thanks, Baodi Shi PengHui Li 于2024年11月8日周五 08:26写道: > > +1 > > I also had many questions during the discussion with Pulsar users. >

Re: [DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-07 Thread PengHui Li
+1 I also had many questions during the discussion with Pulsar users. It's not easy to understand why their consumers get a lot of duplicated messages with batch messages without any consumer or broker restarts. Regards Penghui On Wed, Nov 6, 2024 at 4:45 AM Yunze Xu wrote: > Hi all, > > I wro

[DISCUSS] PIP-391: Enable batch index ACK by default

2024-11-06 Thread Yunze Xu
Hi all, I wrote a PIP (https://github.com/apache/pulsar/pull/23567) to enable batch index ACK by default. >From my experiences, the default ACK semantics are really confusing and it's hard to explain the implementation details to users. Even if it's explained, this design is still not convincing.