Github user zhangminglei commented on a diff in the pull request:
https://github.com/apache/flink/pull/5808#discussion_r179060468
--- Diff:
flink-core/src/main/java/org/apache/flink/configuration/TaskManagerOptions.java
---
@@ -104,40 +104,40 @@
" global ssl flag " +
SecurityOptions.SSL_ENABLED.key() + " is set to true");
/**
- * The initial registration pause between two consecutive registration
attempts. The pause
- * is doubled for each new registration attempt until it reaches the
maximum registration pause.
+ * The initial registration backoff between two consecutive
registration attempts. The backoff
+ * is doubled for each new registration attempt until it reaches the
maximum registration backoff.
*/
- public static final ConfigOption<String> INITIAL_REGISTRATION_PAUSE =
- key("taskmanager.initial-registration-pause")
+ public static final ConfigOption<String> INITIAL_REGISTRATION_BACKOFF =
+ key("taskmanager.registration.initial-backoff")
--- End diff --
I think there already has deprecated key
```"taskmanager.initial-registration-pause"``` in the
```ConfigConstants.java``` file. Below:
```
/**
* The initial registration pause between two consecutive registration
attempts. The pause
* is doubled for each new registration attempt until it reaches the
maximum registration pause.
*
* @deprecated use {@link TaskManagerOptions#INITIAL_REGISTRATION_BACKOFF}
instead
*/
@Deprecated
public static final String TASK_MANAGER_INITIAL_REGISTRATION_PAUSE =
"taskmanager.initial-registration-pause";
```
Or, I am guessing what you mean is that I should change the value
```taskmanager.initial-registration-pause``` to
```taskmanager.registration.initial-backoff``` in ```ConfigConstants.java```
file. But you said ((nor should we change the value)) before. So, Hmm. Am I
missing something ?
---
