zentol closed pull request #7427: [hotfix] [test] Reduce test timeout to a
reasonable value
URL: https://github.com/apache/flink/pull/7427
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
index eb6b3ef8c50..ac4052eceec 100644
---
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
+++
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
@@ -56,8 +56,8 @@
import org.apache.flink.runtime.concurrent.ScheduledExecutorServiceAdapter;
import org.apache.flink.runtime.jobgraph.JobGraph;
import org.apache.flink.runtime.jobgraph.JobStatus;
-import org.apache.flink.runtime.state.AbstractStateBackend;
import org.apache.flink.runtime.state.CheckpointListener;
+import org.apache.flink.runtime.state.StateBackend;
import org.apache.flink.runtime.testingUtils.TestingUtils;
import org.apache.flink.streaming.api.datastream.DataStream;
import org.apache.flink.streaming.api.datastream.QueryableStateStream;
@@ -106,8 +106,8 @@
*/
public abstract class AbstractQueryableStateTestBase extends TestLogger {
- private static final Duration TEST_TIMEOUT = Duration.ofSeconds(10000L);
- public static final long RETRY_TIMEOUT = 50L;
+ private static final Duration TEST_TIMEOUT = Duration.ofSeconds(200L);
+ private static final long RETRY_TIMEOUT = 50L;
private final ScheduledExecutorService executorService =
Executors.newScheduledThreadPool(4);
private final ScheduledExecutor executor = new
ScheduledExecutorServiceAdapter(executorService);
@@ -115,7 +115,7 @@
/**
* State backend to use.
*/
- protected AbstractStateBackend stateBackend;
+ private StateBackend stateBackend;
/**
* Client shared between all the test.
@@ -142,7 +142,7 @@ public void setUp() throws Exception {
*
* @return a state backend instance for each unit test
*/
- protected abstract AbstractStateBackend createStateBackend() throws
Exception;
+ protected abstract StateBackend createStateBackend() throws Exception;
/**
* Runs a simple topology producing random (key, 1) pairs at the
sources (where
@@ -153,7 +153,6 @@ public void setUp() throws Exception {
* to query the counts of each key in rounds until all keys have
non-zero counts.
*/
@Test
- @SuppressWarnings("unchecked")
public void testQueryableState() throws Exception {
final Deadline deadline = Deadline.now().plus(TEST_TIMEOUT);
final int numKeys = 256;
@@ -597,6 +596,7 @@ public Integer getKey(Tuple2<Integer, Long> value) {
}
}).asQueryableState("matata");
+ @SuppressWarnings("unchecked")
final ValueStateDescriptor<Tuple2<Integer, Long>> stateDesc =
(ValueStateDescriptor<Tuple2<Integer, Long>>)
queryableState.getStateDescriptor();
diff --git
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateFsBackendITCase.java
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateFsBackendITCase.java
index 9755b52e2f1..ef6a267df0f 100644
---
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateFsBackendITCase.java
+++
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateFsBackendITCase.java
@@ -25,7 +25,7 @@
import org.apache.flink.configuration.TaskManagerOptions;
import org.apache.flink.configuration.WebOptions;
import org.apache.flink.queryablestate.client.QueryableStateClient;
-import org.apache.flink.runtime.state.AbstractStateBackend;
+import org.apache.flink.runtime.state.StateBackend;
import org.apache.flink.runtime.state.filesystem.FsStateBackend;
import org.apache.flink.runtime.testutils.MiniClusterResourceConfiguration;
import org.apache.flink.test.util.MiniClusterWithClientResource;
@@ -58,7 +58,7 @@
private static MiniClusterWithClientResource miniClusterResource;
@Override
- protected AbstractStateBackend createStateBackend() throws Exception {
+ protected StateBackend createStateBackend() throws Exception {
return new
FsStateBackend(temporaryFolder.newFolder().toURI().toString());
}
diff --git
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateRocksDBBackendITCase.java
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateRocksDBBackendITCase.java
index 94baaed3745..46202c322cb 100644
---
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateRocksDBBackendITCase.java
+++
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/HAQueryableStateRocksDBBackendITCase.java
@@ -26,7 +26,7 @@
import org.apache.flink.configuration.WebOptions;
import org.apache.flink.contrib.streaming.state.RocksDBStateBackend;
import org.apache.flink.queryablestate.client.QueryableStateClient;
-import org.apache.flink.runtime.state.AbstractStateBackend;
+import org.apache.flink.runtime.state.StateBackend;
import org.apache.flink.runtime.testutils.MiniClusterResourceConfiguration;
import org.apache.flink.test.util.MiniClusterWithClientResource;
@@ -58,7 +58,7 @@
private static MiniClusterWithClientResource miniClusterResource;
@Override
- protected AbstractStateBackend createStateBackend() throws Exception {
+ protected StateBackend createStateBackend() throws Exception {
return new
RocksDBStateBackend(temporaryFolder.newFolder().toURI().toString());
}
diff --git
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateFsBackendITCase.java
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateFsBackendITCase.java
index 36637c0d342..f1382556eb7 100644
---
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateFsBackendITCase.java
+++
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateFsBackendITCase.java
@@ -24,7 +24,7 @@
import org.apache.flink.configuration.TaskManagerOptions;
import org.apache.flink.configuration.WebOptions;
import org.apache.flink.queryablestate.client.QueryableStateClient;
-import org.apache.flink.runtime.state.AbstractStateBackend;
+import org.apache.flink.runtime.state.StateBackend;
import org.apache.flink.runtime.state.filesystem.FsStateBackend;
import org.apache.flink.runtime.testutils.MiniClusterResourceConfiguration;
import org.apache.flink.test.util.MiniClusterWithClientResource;
@@ -61,7 +61,7 @@
.build());
@Override
- protected AbstractStateBackend createStateBackend() throws Exception {
+ protected StateBackend createStateBackend() throws Exception {
return new
FsStateBackend(temporaryFolder.newFolder().toURI().toString());
}
diff --git
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateRocksDBBackendITCase.java
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateRocksDBBackendITCase.java
index ce3e665fce6..3ad0dda52e2 100644
---
a/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateRocksDBBackendITCase.java
+++
b/flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/NonHAQueryableStateRocksDBBackendITCase.java
@@ -25,7 +25,7 @@
import org.apache.flink.configuration.WebOptions;
import org.apache.flink.contrib.streaming.state.RocksDBStateBackend;
import org.apache.flink.queryablestate.client.QueryableStateClient;
-import org.apache.flink.runtime.state.AbstractStateBackend;
+import org.apache.flink.runtime.state.StateBackend;
import org.apache.flink.runtime.testutils.MiniClusterResourceConfiguration;
import org.apache.flink.test.util.MiniClusterWithClientResource;
@@ -60,7 +60,7 @@
.build());
@Override
- protected AbstractStateBackend createStateBackend() throws Exception {
+ protected StateBackend createStateBackend() throws Exception {
return new
RocksDBStateBackend(temporaryFolder.newFolder().toURI().toString());
}
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
