[
https://issues.apache.org/jira/browse/FLINK-9042?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16411366#comment-16411366
]
ASF GitHub Bot commented on FLINK-9042:
---------------------------------------
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5736#discussion_r176733032
--- Diff:
flink-tests/src/test/java/org/apache/flink/test/checkpointing/ResumeCheckpointManuallyITCase.java
---
@@ -263,56 +268,113 @@ private void testExternalizedCheckpoints(
config.setString(HighAvailabilityOptions.HA_STORAGE_PATH,
haDir.toURI().toString());
}
- TestingCluster cluster = new TestingCluster(config);
- cluster.start();
+ MiniClusterResource cluster = new MiniClusterResource(
+ new
MiniClusterResource.MiniClusterResourceConfiguration(
+ new Configuration(),
+ NUM_TASK_MANAGERS,
+ SLOTS_PER_TASK_MANAGER),
+ true);
--- End diff --
ha. It _looks_ like it could be, but only because I did a mistake. We're
passing a blank configuration to the cluster, and not the `config` we set up
above.
With that fixed the can't be a class rule since each test requires it's own
configuration. It also cannot be turned into a `Rule` easily as the config
parameters are passed through the methods, and we would have to restructure the
tests rather significantly.
> Port ResumeCheckpointManuallyITCase to flip6
> --------------------------------------------
>
> Key: FLINK-9042
> URL: https://issues.apache.org/jira/browse/FLINK-9042
> Project: Flink
> Issue Type: Improvement
> Components: State Backends, Checkpointing, Tests
> Affects Versions: 1.5.0
> Reporter: Chesnay Schepler
> Assignee: Chesnay Schepler
> Priority: Blocker
> Fix For: 1.5.0
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
