Myasuka commented on code in PR #23347:
URL: https://github.com/apache/flink/pull/23347#discussion_r1360198299


##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/ExecutionCheckpointingOptions.java:
##########
@@ -314,4 +317,28 @@ public class ExecutionCheckpointingOptions {
                             "Defines the maximum number of subtasks that share 
the same channel state file. "
                                     + "It can reduce the number of small files 
when enable unaligned checkpoint. "
                                     + "Each subtask will create a new channel 
state file when this is configured to 1.");
+
+    public static final ConfigOption<ThreadDumpLogLevel>
+            THREAD_DUMP_LOG_LEVEL_WHEN_CHECKPOINT_TIMEOUT =
+                    
key("execution.checkpointing.thread-dump-log-level-when-checkpoint-timeout")

Review Comment:
   First of all, I think not all scenarios obey the `execution.checkpointing` 
prefix. And I think making different log level for different scenarios would be 
a bit too complex. We only need two parameters no matter how many scenarios 
supported in my proposal.



-- 
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-unsubscr...@flink.apache.org

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

Reply via email to