ableegoldman edited a comment on pull request #10608: URL: https://github.com/apache/kafka/pull/10608#issuecomment-829528563
I'm going to merge this now since Walker approved it (thanks!) and I'm waiting to rebase the NamedTopology PR on top of these changes. If anyone comes across this and has any concerns, I'm happy to discuss and/or submit a followup PR if necessary ( to make it clear, committer review still welcome though it was merged) edit: I should have waited for a committer review on this, I was thinking of it as trivial (originally I was just going to remove a single parameter on two lines) but clearly it grew to be more substantial. Apologies -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org