Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5364#discussion_r165459894
--- Diff:
flink-tests/src/test/java/org/apache/flink/test/state/operator/restore/keyed/KeyedJob.java
---
@@ -100,9 +100,7 @@ public static void main(String[] args) throws Exception
{
.map(new StatefulStringStoringMap(mode, "first"))
.setParallelism(4);
- if (mode == ExecutionMode.MIGRATE || mode ==
ExecutionMode.RESTORE) {
--- End diff --
yeah it should be alright to remove that, but let's chain the uid call to
the operator creation as we do for the others for style points.
---
