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


Reply via email to