Hi Ming-Yen, Thanks for the KIP!
A1: I have a question on how the internals will be adapted to use `ConfigEntity#entityName()` instead of `ConfigEntity#name()`. For instance, `ClientQuotaManager#clientId()` and `ClientQuotaManager#sanitizedUser()`rely on `ConfigEntity#name()` and expect a String value to create metric tags for the quota metrics. Will those continue to be artificially mapped to the original `"<default>"` string for compatibility? Best, Alyssa On Wed, Jun 3, 2026 at 2:29 AM Ming-Yen Chung <[email protected]> wrote: > Hi all, > > I would like to start a discussion on KIP-1341: Replace <default> > placeholder in ConfigEntity#name() with Optional<String> entityName() > < > https://urldefense.com/v3/__https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=421958794__;!!Ayb5sqE7!qh86EDHNNNUNMHY54gAc4iUo0Bj2Sa3iJo6kDxAI-t6C7HGiGMPV81Um1OhhMNaPLXTPKVBZ9YNrp2mzmy_E_Q$ > > > > ConfigEntity#name() returns "<default>" for default quota entities, while > every other public API uses null. The "<default>" value is a leftover from > the ZooKeeper era and has no meaning in KRaft. This KIP adds > Optional<String> entityName() and deprecates name(). > > Thanks, > Ming-Yen >
