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

Guozhang Wang commented on KAFKA-14130:
---------------------------------------

Hello [~hoping][~Eslam.hamdy] sorry about the confusion.. I have complete this 
ticket myself through this PR: https://github.com/apache/kafka/pull/12476. I 
forgot to close this ticket though, which is my bad.. closing now.

> Reduce RackAwarenessIntegrationTest to a unit test
> --------------------------------------------------
>
>                 Key: KAFKA-14130
>                 URL: https://issues.apache.org/jira/browse/KAFKA-14130
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams, unit tests
>            Reporter: Guozhang Wang
>            Assignee: Eslam
>            Priority: Major
>              Labels: newbie
>
> While working on KAFKA-13877, I feel it's an overkill to introduce the whole 
> test class as an integration test, since all we need is to just test the 
> assignor itself which could be a unit test. Running this suite with 9+ 
> instances takes long time and is still vulnerable to all kinds of timing 
> based flakiness. A better choice is to reduce it as a unit test, similar to 
> {{HighAvailabilityStreamsPartitionAssignorTest}} that just test the behavior 
> of the assignor itself, rather than creating many instances hence depend on 
> various timing bombs to not explode.
> The scope of this ticket is to refactor the {{RackAwarenessIntegrationTest}} 
> into a {{RackAwarenessStreamsPartitionAssignorTest}}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to