Thanks for the KIP Rohit.

Wondering, if we should also add `TopologyTestDriver(Topology,
Instant)`? Not totally sure, as having too many overload could also be
annoying.


-Matthias

On 11/3/20 10:02 AM, Rohit Deshpande wrote:
> Hello all,
> I have created KIP-680: TopologyTestDriver should not require a Properties
> argument.
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-680%3A+TopologyTestDriver+should+not+require+a+Properties+argument
> 
> Jira for the KIP:
> https://issues.apache.org/jira/browse/KAFKA-10629
> 
> If we end up making changes, they will look like this:
> https://github.com/apache/kafka/compare/trunk...rohitrmd:KAFKA-10629
> 
> Please have a look and let me know what you think.
> 
> Thanks,
> Rohit
> 

Reply via email to