dawidwys commented on a change in pull request #14235:
URL: https://github.com/apache/flink/pull/14235#discussion_r533572934
##########
File path: docs/dev/execution_configuration.md
##########
@@ -50,9 +50,9 @@ With the closure cleaner disabled, it might happen that an
anonymous user functi
- `getMaxParallelism()` / `setMaxParallelism(int parallelism)` Set the default
maximum parallelism for the job. This setting determines the maximum degree of
parallelism and specifies the upper limit for dynamic scaling.
-- `getNumberOfExecutionRetries()` / `setNumberOfExecutionRetries(int
numberOfExecutionRetries)` Sets the number of times that failed tasks are
re-executed. A value of zero effectively disables fault tolerance. A value of
`-1` indicates that the system default value (as defined in the configuration)
should be used. This is deprecated, use [restart strategies]({{ site.baseurl
}}/dev/restart_strategies.html) instead.
+- `getNumberOfExecutionRetries()` / `setNumberOfExecutionRetries(int
numberOfExecutionRetries)` Sets the number of times that failed tasks are
re-executed. A value of zero effectively disables fault tolerance. A value of
`-1` indicates that the system default value (as defined in the configuration)
should be used. This is deprecated, use [restart strategies]({% link
redirects/restart_strategies.md %}) instead.
-- `getExecutionRetryDelay()` / `setExecutionRetryDelay(long
executionRetryDelay)` Sets the delay in milliseconds that the system waits
after a job has failed, before re-executing it. The delay starts after all
tasks have been successfully stopped on the TaskManagers, and once the delay is
past, the tasks are re-started. This parameter is useful to delay re-execution
in order to let certain time-out related failures surface fully (like broken
connections that have not fully timed out), before attempting a re-execution
and immediately failing again due to the same problem. This parameter only has
an effect if the number of execution re-tries is one or more. This is
deprecated, use [restart strategies]({{ site.baseurl
}}/dev/restart_strategies.html) instead.
+- `getExecutionRetryDelay()` / `setExecutionRetryDelay(long
executionRetryDelay)` Sets the delay in milliseconds that the system waits
after a job has failed, before re-executing it. The delay starts after all
tasks have been successfully stopped on the TaskManagers, and once the delay is
past, the tasks are re-started. This parameter is useful to delay re-execution
in order to let certain time-out related failures surface fully (like broken
connections that have not fully timed out), before attempting a re-execution
and immediately failing again due to the same problem. This parameter only has
an effect if the number of execution re-tries is one or more. This is
deprecated, use [restart strategies]({% link redirects/restart_strategies.md
%}) instead.
Review comment:
We should not link to redirects. Redirects are only for pages that do no
longer exist.
----------------------------------------------------------------
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
