cadonna commented on code in PR #12191:
URL: https://github.com/apache/kafka/pull/12191#discussion_r880435973
##########
connect/runtime/src/test/java/org/apache/kafka/connect/integration/BlockingConnectorTest.java:
##########
@@ -78,8 +79,8 @@ public class BlockingConnectorTest {
private static final String NORMAL_CONNECTOR_NAME = "normal-connector";
private static final String TEST_TOPIC = "normal-topic";
private static final int NUM_RECORDS_PRODUCED = 100;
- private static final long CONNECT_WORKER_STARTUP_TIMEOUT =
TimeUnit.SECONDS.toMillis(60);
- private static final long RECORD_TRANSFER_DURATION_MS =
TimeUnit.SECONDS.toMillis(30);
+ private static final long CONNECTOR_BLOCK_TIMEOUT_MS =
TimeUnit.MINUTES.toMillis(2);
+ private static final long RECORD_TRANSFER_TIMEOUT_MS =
TimeUnit.MINUTES.toMillis(2);
Review Comment:
Have you tried just doubling the original 30 seconds to 1 minute?
##########
connect/runtime/src/test/java/org/apache/kafka/connect/util/clusters/EmbeddedConnectClusterAssertions.java:
##########
@@ -44,9 +44,9 @@
public class EmbeddedConnectClusterAssertions {
private static final Logger log =
LoggerFactory.getLogger(EmbeddedConnectClusterAssertions.class);
- public static final long WORKER_SETUP_DURATION_MS =
TimeUnit.SECONDS.toMillis(60);
+ public static final long WORKER_SETUP_DURATION_MS =
TimeUnit.MINUTES.toMillis(5);
Review Comment:
This seems quite an increase. Have you tried with 2 minutes?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
