[GitHub] [kafka] cmccabe commented on pull request #13337: MINOR: Refactor the MetadataPublisher interface

2023-03-09 Thread via GitHub
cmccabe commented on PR #13337: URL: https://github.com/apache/kafka/pull/13337#issuecomment-1462530266 I fixed the case where I was invoking `DynamicConfigPublisher.publish` with a null. This was a bit silly... I was just being lazy about adding an extra function. I also fixed it

[GitHub] [kafka] cmccabe commented on pull request #13337: MINOR: Refactor the MetadataPublisher interface

2023-03-07 Thread via GitHub
cmccabe commented on PR #13337: URL: https://github.com/apache/kafka/pull/13337#issuecomment-1458906783 rebase on trunk -- 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