FrankYang0529 commented on PR #18685: URL: https://github.com/apache/kafka/pull/18685#issuecomment-2705314828
> It looks like we both created PRs around the same time to fix this bug. Mine is here : #19127 . If it's ok to remove `FeaturesPublisher`, I think we can keep using this PR. I have add your test case `KRaftClusterTest#testDescribeKRaftVersion` to this PR and add you as co-author. > While I understand why you did it, moving kraft.version out of BootstrapMetadata in tests feels messy to me. I think we should leave this alone for now and think about it more in the future. Perhaps surfacing the initial raft state in BootstrapMetadata.java is possible. That would be better. Agree, I revert the commit. I will create a Jira for followup. Thanks. -- 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
