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 > > > > >
