[ 
https://issues.apache.org/jira/browse/KAFKA-6474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16749129#comment-16749129
 ] 

Filipe Agapito commented on KAFKA-6474:
---------------------------------------

Hi [~mjsax]. Thanks for pinging me. Sorry, I haven't been able to contribute 
lately.

I have an open PR which removes a few more instances of KStreamTest driver. I 
also have some work done on the remaining classes which use KStreamTestDriver 
(only two remain after that PR, if I'm not mistaken).

To avoid duplication of efforts, if you will give me just 1 or 2 days more I 
can fix and submit my work. I think [https://github.com/apache/kafka/pull/5433] 
should be easy enough to fix. I'll just rebase it, fix the conflicts, and make 
the changes you suggested (I'm doing that now).

If you can't spare the additional couple of days and this is blocking you in 
any way I'm ok with you proceeding as discussed in the other ticket.

 

> Rewrite test to use new public TopologyTestDriver
> -------------------------------------------------
>
>                 Key: KAFKA-6474
>                 URL: https://issues.apache.org/jira/browse/KAFKA-6474
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams, unit tests
>    Affects Versions: 1.1.0
>            Reporter: Matthias J. Sax
>            Assignee: Filipe Agapito
>            Priority: Major
>              Labels: newbie
>
> With KIP-247 we added public TopologyTestDriver. We should rewrite out own 
> test to use this new test driver and remove the two classes 
> ProcessorTopoogyTestDriver and KStreamTestDriver.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to