hi,

We should ensure forward and backward compatibility between 3.0 and
4.0. We can enable
configuration(managedLedgerPersistIndividualAckAsLongArray) in 5.0.

Therefore, I believe the default value should be false.

We shouldn't be constrained by the already released 4.0.x versions. I
believe users prefer to choose the newer 4.0.x versions and have
seamless switching with 3.0.x.


Thanks,
Baodi Shi

Yubiao Feng <yubiao.f...@streamnative.io.invalid> 于2025年1月13日周一 09:50写道:
>
> Hi Lari
>
> > We have already made releases 4.0.0 and 4.0.1
> > with the changed defaults. Changing it now is too late.
>
> I think it is a fault and we should correct it, So let set the default
> value to `false`.
>
> > PR 9292 has been approved by the community and there
> > have been discussions in the past.
>
> I have never seen there is a discussion in the email list, and it was
> merged with a unique approvement.
>
> Since it already introduced a regression, and no others mentioned how to
> upgrade/downgrade gracefully so far, we should set the default value to
> `false`.
>
> On Thu, Jan 9, 2025 at 9:15 PM Lari Hotari <lhot...@apache.org> wrote:
>
> > On Thu, 9 Jan 2025 at 14:26, Yubiao Feng
> > <yubiao.f...@streamnative.io.invalid> wrote:
> > > > Isn't the typical way to handle regressions and bugs to fix them?
> > >
> > > It may have other potential issues.
> > >
> > > > I don't see a sensible argument for changing configuration defaults for
> > > 4.0.2 .
> > >
> > > Is there a PIP related to #9292, which defined the default behavior has
> > > been changed?
> >
> > We have already made releases 4.0.0 and 4.0.1 with the changed
> > defaults. Changing it now is too late.
> > PR 9292 has been approved by the community and there have been
> > discussions in the past.
> > I don't see why we would be postponing such an important improvement
> > to Pulsar 5.0 . Let's fix any possible regressions asap and adapt.
> >
> > -Lari
> >

Reply via email to