tillrohrmann commented on a change in pull request #15153:
URL: https://github.com/apache/flink/pull/15153#discussion_r593272810



##########
File path: 
flink-tests/src/test/java/org/apache/flink/test/scheduling/PipelinedRegionSchedulingITCase.java
##########
@@ -204,4 +248,22 @@ public void invoke() throws Exception {
             }
         }
     }
+
+    /** Invokable which fails exactly once with a {@link PartitionException}. 
*/
+    public static class OneTimeFailingReceiverWithPartitionException extends 
AbstractInvokable {
+
+        private static final AtomicBoolean hasFailed = new 
AtomicBoolean(false);

Review comment:
       I would suggest to make this resettable. Otherwise it won't be possible 
to run `testRecoverFromPartitionException` repeatedly from your IDE. Only the 
first run would throw the `PartitionNotFoundException` and all other runs would 
simply be a `NoOpInvokable`.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to