[ 
https://issues.apache.org/jira/browse/KAFKA-18919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17932685#comment-17932685
 ] 

Ismael Juma commented on KAFKA-18919:
-------------------------------------

Perhaps. I think it would be a good idea to have a KIP where we can make sure 
we are not missing any important details. If we agree that the compatibility 
risk is indeed acceptable for a feature release, we can perhaps do it in 4.1.

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

Reply via email to