[GitHub] [flink] RocMarshal commented on a diff in pull request #21999: [FLINK-29816][streaming] Fix the bug that SteamTask skip handle exception during restoring.

["RocMarshal (via GitHub)" <gi...@apache.org>]

RocMarshal commented on code in PR #21999:
URL: https://github.com/apache/flink/pull/21999#discussion_r1114607304


##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java:
##########
@@ -261,6 +261,12 @@
      */
     private volatile boolean isRunning;
 
+    /**
+     * Flag to mark the task "in operation", in which case check needs to be process the exceptions
+     * thrown in the initializing duration.
+     */
+    private volatile boolean isInitializing;

Review Comment:
   Thanks for the clarify.  I got it.
   We need to define or describe different states for  different abstract level as accurately as possible, right?
   I'd prefer your description `isRestoring`, now.



-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org