pprovenzano commented on PR #14083: URL: https://github.com/apache/kafka/pull/14083#issuecomment-1651985826
> I just took a quick look and I can see multiple protocol change, quite a big refactor to make it working with KRaft and it feels like we should write a KIP about this. What do you think? There isn't an API change for using the the Kafka client API so customers shouldn't need to change how they use delegation tokens so we are covered there. The conversion of using Zk to using metadata should be covered with [KIP-833](https://cwiki.apache.org/confluence/display/KAFKA/KIP-746%3A+Revise+KRaft+Metadata+Records). Updating the Inter Broker Protocol with new records is allowed with a bump in the IBP version which is included in the PR. I bumped it from IBP_3_6_IV0 to IBP_3_6_IV1 similar to what I did for Scram. So, I think we are covered from any KIP requirements. @cmccabe can you chime in here? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
