Hi TaiJuWu, Thanks for the KIP.
LC1: From the rejective alternative: "Reporting all non-sensitive configs (~388 entries, ~18 KB per broker) was considered. While simpler in filtering logic, it includes 163 configs that have no ConfigDef validator. The additional ~8 KB per broker adds up in large clusters (8 MB for 1000 brokers) with no practical benefit — configs without validators cannot be validated regardless of whether they are reported. The chosen approach (only configs with validators, ~225 entries, ~10 KB per broker) keeps the payload smaller and automatically expands when new validators are added in future KIPs." So, ~ 10 KB per broker now, and in 1000 brokers cluster, it'll be 10MB metadata log increase, right? I'm not convinced that this is necessary after reading the KIP. If you think 18 KB per broker is "big", then I don't know why 10 KB per broker is acceptable? Thanks, Luke On Thu, Jun 4, 2026 at 4:34 PM TaiJu Wu <[email protected]> wrote: > Hi chia: > > Thanks for your feedback. > > chia_00: I have removed this section, we don't change ConfigDef and add > test plan to cover server-side filter helper. > chia_01: thanks for catching this, I have updated KIP. > chia_02: I added following description to this KIP. > If a broker reports an empty StaticConfigs, the controller should reject > the registration and prevent the broker from joining the cluster by > returning *INVALID_REGISTRATION*. > > Thanks, > TaiJuWu > > > > Chia-Ping Tsai <[email protected]> 於 2026年6月1日週一 上午11:05寫道： > > > chia_2: should the controller reject the request which has empty static > > configs? > > > > On 2026/04/30 14:47:05 TaiJu Wu wrote: > > > Hi all, > > > > > > I would like to discuss KIP-1326 > > > < > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1326%3A+Include+broker+static+configurations+in+BrokerRegistrationRequest > > > > > > which > > > proposes that brokers report a subset of their static configuration > > values > > > to the controller as part of BrokerRegistrationRequest. > > > > > > WIKI: > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1326%3A+Include+broker+static+configurations+in+BrokerRegistrationRequest > > > > > > Best, > > > TaiJuWu > > > > > >
