kkonstantine commented on a change in pull request #8928:
URL: https://github.com/apache/kafka/pull/8928#discussion_r449789475
##########
File path:
connect/runtime/src/test/java/org/apache/kafka/connect/integration/BlockingConnectorTest.java
##########
@@ -76,6 +78,15 @@ public void setup() {
// start the clusters
connect.start();
+
+ // wait for the Connect REST API to become available. necessary
because of the reduced REST
+ // request timeout; otherwise, we may get an unexpected 500 with our
first real REST request
+ // if the worker is still getting on its feet.
+ waitForCondition(
Review comment:
Should we change the assertions then? I'd assume this will be useful to
other tests as well.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
