jiexray commented on code in PR #21410:
URL: https://github.com/apache/flink/pull/21410#discussion_r1094665507
##########
flink-runtime/src/test/java/org/apache/flink/runtime/state/FileStateBackendMigrationTest.java:
##########
@@ -18,18 +18,24 @@
package org.apache.flink.runtime.state;
import org.apache.flink.runtime.state.filesystem.FsStateBackend;
+import org.apache.flink.testutils.junit.extensions.parameterized.Parameters;
-import java.io.File;
+import java.util.Arrays;
+import java.util.Collection;
/**
* Tests for the keyed state backend and operator state backend, as created by
the {@link
* FsStateBackend}.
*/
public class FileStateBackendMigrationTest extends
StateBackendMigrationTestBase<FsStateBackend> {
+ @Parameters
+ public static Collection<Object> modes() {
+ return Arrays.asList("just to please junit5 parameter test in base
class");
Review Comment:
I have changed the expression in the parameter list to enhanace reabability.
```
@Parameters
public static Collection<Object> modes() {
// current test does not need to be parameterized
return Collections.singletonList("no-use");
}
```
--
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]
