Hello all, I've updated KIP-1256 to include a new mechanism for broker static config reporting. The core idea is that brokers will send their relevant static configurations to the controller during registration via BrokerRegistrationRequest. This is necessary because some dynamic config validations on the broker side depend on static config context — without this information, the controller cannot fully replicate the broker's validation logic. With this change, the controller can perform the same context-aware validation as brokers, completing the alignment addressed in this KIP.
Any feedback would be appreciated. Best Regards, Jiunn-Yang > 黃竣陽 <[email protected]> 於 2026年2月12日 晚上10:15 寫道： > > Hello José, > > Thanks you for your comments, > > I have updated the Public Interfaces section to note that this KIP does not > change any public interfaces, > and also revised the Proposed Changes section to describe the behavior > changes instead of relying on code. > > Best Regards, > Jiunn-Yang > >> José Armando García Sancio via dev <[email protected]> 於 2026年2月12日 >> 凌晨1:59 寫道： >> >> Hi Jiunn-Yang, >> >> Thanks very much for looking into these inconsistencies, thoroughly >> documenting them and proposing a fix. >> >> This is minor but in the public interface section you have: "Update >> the processing logic in ControllerApis#handleIncrementalAlterConfigs." >> Technically, that method is not part of Kafka's public interface. This >> KIP doesn't change any public interfaces. It is okay to just state >> that in the public interfaces section. >> >> In the proposed changes section you include code for the change. It is >> difficult to evaluate the code since it is not clear to me what is the >> implementation for some of the symbols included. In the motivation >> section you enumerated all of the inconsistency between the broker and >> the controller. What do you think of adding a similar enumeration in >> the proposed changes section explaining how you plan to resolve those >> inconsistencies? >> >> Thanks, >> -- >> -José >
