You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by tr...@apache.org on 2020/05/22 16:49:49 UTC
[flink] branch release-1.10 updated: [FLINK-17801][tests] Increase
timeout of TaskExecutorTest.testHeartbeatTimeoutWithResourceManager
This is an automated email from the ASF dual-hosted git repository.
trohrmann pushed a commit to branch release-1.10
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.10 by this push:
new 7132038 [FLINK-17801][tests] Increase timeout of TaskExecutorTest.testHeartbeatTimeoutWithResourceManager
7132038 is described below
commit 713203808b54050431d687dbf1d524c900c7141b
Author: Till Rohrmann <tr...@apache.org>
AuthorDate: Tue May 19 10:53:40 2020 +0200
[FLINK-17801][tests] Increase timeout of TaskExecutorTest.testHeartbeatTimeoutWithResourceManager
Increasing the timeout because on AZP it can happen that calling the disconnectTaskExecutor takes
more than 150 ms. This commit increases the timeout to 10s.
This closes #12242.
---
.../java/org/apache/flink/runtime/taskexecutor/TaskExecutorTest.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/taskexecutor/TaskExecutorTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/taskexecutor/TaskExecutorTest.java
index 9feb91b..f0dbbd1 100644
--- a/flink-runtime/src/test/java/org/apache/flink/runtime/taskexecutor/TaskExecutorTest.java
+++ b/flink-runtime/src/test/java/org/apache/flink/runtime/taskexecutor/TaskExecutorTest.java
@@ -432,7 +432,7 @@ public class TaskExecutorTest extends TestLogger {
assertThat(taskExecutorRegistrationFuture.get(timeout.toMilliseconds(), TimeUnit.MILLISECONDS), equalTo(taskManagerLocation.getResourceID()));
// heartbeat timeout should trigger disconnect TaskManager from ResourceManager
- assertThat(taskExecutorDisconnectFuture.get(heartbeatTimeout * 50L, TimeUnit.MILLISECONDS), equalTo(taskManagerLocation.getResourceID()));
+ assertThat(taskExecutorDisconnectFuture.get(timeout.toMilliseconds(), TimeUnit.MILLISECONDS), equalTo(taskManagerLocation.getResourceID()));
assertTrue(
"The TaskExecutor should try to reconnect to the RM",