Hi Edoardo, Thanks for the update. Looks good to me.
Luke On Tue, Dec 9, 2025 at 7:20 PM Edoardo Comar <[email protected]> wrote: > KIP amended with > ` > The new configuration entry will not be ported to 4.x versions where > there is no ZkAdminManager component > ` > > On Mon, 8 Dec 2025 at 11:33, Luke Chen <[email protected]> wrote: > > > > Thanks Edoardo. > > I'm +1 (binding) as long as we make it clear in the KIP. > > > > > > Luke > > > > On Mon, Dec 8, 2025 at 8:30 PM Edoardo Comar <[email protected]> > wrote: > > > > > Thanks Luke, > > > I can add that this is not ever to be ported to 4.x > > > > > > On Mon, 8 Dec 2025 at 06:54, Luke Chen <[email protected]> wrote: > > > > > > > > Hi Edoardo, > > > > > > > > Thanks for the KIP! Nice find and fix! > > > > Overall LGTM, just one minor comment: > > > > > > > > I think we should explicitly mention the config will be removed in > v4.0 > > > in > > > > the "Compatibility" section. > > > > > > > > Thank you, > > > > Luke > > > > > > > > On Mon, Dec 8, 2025 at 3:49 PM Chia-Ping Tsai <[email protected]> > > > wrote: > > > > > > > > > +1 (binding) > > > > > > > > > > > > > > > PoAn Yang <[email protected]> 於 2025年12月8日週一 下午2:30寫道： > > > > > > > > > > > Hi Edoardo, > > > > > > > > > > > > Thanks for the mini-KIP. > > > > > > > > > > > > It's good that we have a compromise solution that allows us to > keep > > > > > > the existing behavior while providing an opportunity to align the > > > > > behavior. > > > > > > > > > > > > +1 (binding) > > > > > > > > > > > > Kind Regards, > > > > > > PoAn > > > > > > > > > > > > > On Dec 5, 2025, at 9:11 PM, Edoardo Comar < > [email protected]> > > > > > wrote: > > > > > > > > > > > > > > Hi All, > > > > > > > as already discussed in the separate mailing list thread > > > > > > > [DISCUSS] AlterConfigPolicy behavior in 3.9.2 > > > > > > > > https://lists.apache.org/thread/bknho1vrvby1tpq634g744vqf4mr8coh > > > > > > > > > > > > > > I would like to propose this minikip > > > > > > > KIP-1252: AlterConfigPolicy compatibility in legacy Zookeeper > mode > > > > > > > (3.9.2 release) > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=399279475 > > > > > > > > > > > > > > that aims to combine a fix to an old inconsistency without > breaking > > > > > > > any backward compatibility. > > > > > > > > > > > > > > Adding new config in minor releases is an exception, > > > > > > > as for ZK support the 3.9.x branch is the end of the line > > > > > > > > > > > > > > Objections are welcome > > > > > > > but If no objections are raised, you might even cast a vote in > this > > > > > > thread :-) > > > > > > > > > > > > > > cheers, > > > > > > > Edo > > > > > > > > > > > > > > > > > > > > >
