jolshan commented on PR #14721: URL: https://github.com/apache/kafka/pull/14721#issuecomment-1803010902
> None of the other changes can wind up in an API response. So for compatibility, we should drop the DymamicBrokerConfig change, but the others should be safe to keep. If we want to be extra safe about it, the only one I actually care about is SslFactory, I'm happy to drop the other changes. I'd be happy with just removing the DynamicBrokerConfig change if we feel good about the others not causing issues. -- 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
