lkokhreidze commented on a change in pull request #11837:
URL: https://github.com/apache/kafka/pull/11837#discussion_r827960149



##########
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:
       Thanks @cadonna 
   If no objections, I will do that now. The fact that it times out in 1 minute 
doesn't give much confidence that it won't be flaky.
   I will follow up the system test.
   fyi @showuon 




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


Reply via email to