cadonna commented on a change in pull request #11837:
URL: https://github.com/apache/kafka/pull/11837#discussion_r827776021
##########
File path:
streams/src/test/java/org/apache/kafka/streams/integration/RackAwarenessIntegrationTest.java
##########
@@ -108,55 +117,122 @@ public void cleanup() throws IOException {
kafkaStreamsInstances.clear();
}
+ @Test
+ public void
shouldThrowConfigExceptionWhenRackAwareAssignmentTagsExceedTheLimit() {
Review comment:
I see. I asked some other people and apparently the size of the
subscription is not that much of a concern currently. I would still propose to
keep the test with 100 instead of 200 tasks. With 200 tasks, the test takes ~10
s on my laptop whereas with 100 tasks it takes ~6 s. I think 6 s is acceptable
and might also reduce flakiness. @lkokhreidze and @showuon WDYT?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
