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

Ning Zhang commented on KAFKA-10304:
------------------------------------

Hi [~mimaison] [~ryannedolan] this is mostly a refactoring pr on the MM2 
integration tests. The purpose of doing that: (1) address the concern in the 
previous PR (https://github.com/apache/kafka/pull/9029), (2) prepare for the 
future development (e.g. extract common functions). I think the current PR 
(https://github.com/apache/kafka/pull/9224) is just a starting point, and I am 
very appreciated for your feedback on what to test additionally and how to get 
close to the real scenario.

> Revisit and improve the tests of MirrorMaker 2
> ----------------------------------------------
>
>                 Key: KAFKA-10304
>                 URL: https://issues.apache.org/jira/browse/KAFKA-10304
>             Project: Kafka
>          Issue Type: Test
>          Components: KafkaConnect, mirrormaker
>            Reporter: Ning Zhang
>            Assignee: Ning Zhang
>            Priority: Minor
>             Fix For: 2.8.0
>
>
> In a different MM2 change https://github.com/apache/kafka/pull/9029, some 
> concerns on tests were raised. It may be a good time to revisit and refactor 
> the tests, possibly in the following way:
> (1) are 100 messages good enough for integration tests?
> (2) what about the broker failure in the middle of integration tests?
> (3) other validations to check (e.g. topic config sync....)



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to