[
https://issues.apache.org/jira/browse/KAFKA-18919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17932672#comment-17932672
]
Ismael Juma commented on KAFKA-18919:
-------------------------------------
Why would we document it versus having `KafkaPrincipalBuilder` extend
`KafkaPrincipalSerde`? This was not possible while we still supported ZK, but
seems like a much better solution now that we no longer support ZK - it's a lot
less error prone.
> Clarify that KafkaPrincipalBuilder classes must also implement
> KafkaPrincipalSerde
> ----------------------------------------------------------------------------------
>
> Key: KAFKA-18919
> URL: https://issues.apache.org/jira/browse/KAFKA-18919
> Project: Kafka
> Issue Type: Sub-task
> Reporter: Mickael Maison
> Assignee: Szu-Yung Wang
> Priority: Major
> Fix For: 4.0.0
>
>
> In KRaft, custom KafkaPrincipalBuilder implementations must also implement
> KafkaPrincipalSerde, otherwise brokers are not able to forward requests to
> the controller.
> So we need to update our docs.
> -We should also be able to do some cleanup in the code. For example
> RequestContext uses Optional<KafkaPrincipalSerde>, we should be able to
> remove Optional since it's now always present.- moved to KAFKA-18922
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
