Github user tzulitai commented on a diff in the pull request:
https://github.com/apache/flink/pull/5230#discussion_r163936390
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/state/OperatorStateHandle.java
---
@@ -36,8 +36,9 @@
* The modes that determine how an {@link OperatorStateHandle} is
assigned to tasks during restore.
*/
public enum Mode {
- SPLIT_DISTRIBUTE, // The operator state partitions in the state
handle are split and distributed to one task each.
- BROADCAST // The operator state partitions are broadcast to all
task.
+ SPLIT_DISTRIBUTE, // The operator state partitions in the
state handle are split and distributed to one task each.
+ BROADCAST, // The operator state
partitions are broadcasted to all tasks.
+ UNIFORM_BROADCAST // The operator states are identical,
and they are broadcasted to all tasks.
--- End diff --
nit: can we either keep with spaces here, or at least tab them so that the
3 comments are aligned?
---
