[ 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)