zhuzhurk commented on a change in pull request #9113: [FLINK-13222] [runtime]
Add documentation for failover strategy option
URL: https://github.com/apache/flink/pull/9113#discussion_r303838329
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/JobManagerOptions.java
##########
@@ -106,17 +106,21 @@
/**
* This option specifies the failover strategy, i.e. how the job
computation recovers from task failures.
*/
- @Documentation.ExcludeFromDocumentation("The failover strategy feature
is highly experimental.")
public static final ConfigOption<String> EXECUTION_FAILOVER_STRATEGY =
key("jobmanager.execution.failover-strategy")
.defaultValue("full")
.withDescription(Description.builder()
- .text("This option specifies how the job
computation recovers from task failures. " +
+ .text("This option specifies the failover
strategy, i.e. " +
+ "how the job computation recovers from
task failures. " +
"Accepted values are:")
.list(
- text("'full': Restarts all tasks."),
- text("'individual': Restarts only the
failed task. Should only be used if all tasks are independent components."),
- text("'region': Restarts all tasks that
could be affected by the task failure.")
+ text("'full': Use RestartAllStrategy to
restarts all tasks to recover the job."),
+ text("'region': Use
AdaptedRestartPipelinedRegionStrategyNG to restart all tasks " +
Review comment:
Just feel that adding a "Failover Strategies" section in "Restart
Strategies" might be a bit confusing to users, indicating that failover
strategy is part of restart strategy.
Would it be better to re-org the page as below?
**Task Failure Recovery**
+- Restart Strategies
+- Fixed Delay Restart Strategy
+- Failure Rate Restart Strategy
+- ...
+- Failover Strategies
+- Restart All Strategy
+- Restart Pipelined Region Strategy
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
