[ 
https://issues.apache.org/jira/browse/FLINK-15813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17110130#comment-17110130
 ] 

Gary Yao commented on FLINK-15813:
----------------------------------

This issue is probably only for the documentation page and doesn't require 
changing tests.

{{FailoverStrategyFactoryLoader}} already uses "region" as the default value:
{code}
                // the default NG failover strategy is the region failover 
strategy.
                // TODO: Remove the overridden default value when removing 
legacy scheduler
                //  and change the default value of 
JobManagerOptions.EXECUTION_FAILOVER_STRATEGY
                //  to be "region"
                final String strategyParam = config.getString(
                        JobManagerOptions.EXECUTION_FAILOVER_STRATEGY,
                        PIPELINED_REGION_RESTART_STRATEGY_NAME);
{code}

> Set default value of jobmanager.execution.failover-strategy to region
> ---------------------------------------------------------------------
>
>                 Key: FLINK-15813
>                 URL: https://issues.apache.org/jira/browse/FLINK-15813
>             Project: Flink
>          Issue Type: Improvement
>          Components: Runtime / Coordination
>    Affects Versions: 1.10.0
>            Reporter: Till Rohrmann
>            Assignee: Zhu Zhu
>            Priority: Blocker
>              Labels: pull-request-available, usability
>             Fix For: 1.11.0
>
>
> We should set the default value of {{jobmanager.execution.failover-strategy}} 
> to {{region}}. This might require to adapt existing tests to make them pass.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to