flashmouse commented on code in PR #13920:
URL: https://github.com/apache/kafka/pull/13920#discussion_r1274483457
##########
clients/src/test/java/org/apache/kafka/clients/consumer/internals/AbstractStickyAssignorTest.java:
##########
@@ -724,6 +725,90 @@ public void
testLargeAssignmentAndGroupWithNonEqualSubscription(boolean hasConsu
assignor.assignPartitions(partitionsPerTopic, subscriptions);
}
+ @Timeout(90)
Review Comment:
Oh, this 90 should be sec, I set 90 sec because in my m1 air this test may
run about 25 sec or more time.
I tried ``TestUtils.waitForCondition`` but seems it is not effective and
couldn't throw Exception when execution use time more than expected, I'm not
familiar with this and couldn't find the reason.
--
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
