Hmm, why are we passing the whole config when we only need two parameters? One thing we should consider is introducing a `RollParams` case class to make this more maintainable. Generally, it makes it harder to test and understand if we pass these config classes with lots of params when we only need a subset.
[ Full content available at: https://github.com/apache/kafka/pull/5728 ] This message was relayed via gitbox.apache.org for devnull@infra.apache.org
