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 >
