Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/2707#discussion_r86104959
--- Diff:
flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/operators/CheckpointCommitter.java
---
@@ -98,17 +89,19 @@ public void setOperatorSubtaskId(int id) throws
Exception {
/**
* Mark the given checkpoint as completed in the resource.
*
- * @param checkpointID
+ * @param subtaskIdx the index of the subtask responsible for
committing the checkpoint.
+ * @param checkpointID this id of the checkpoint to be committed.
* @throws Exception
*/
- public abstract void commitCheckpoint(long checkpointID) throws
Exception;
+ public abstract void commitCheckpoint(int subtaskIdx, long
checkpointID) throws Exception;
/**
* Checked the resource whether the given checkpoint was committed
completely.
*
- * @param checkpointID
+ * @param subtaskIdx the index of the subtask responsible for
committing the checkpoint.
+ * @param checkpointID this id of the checkpoint we are interested in.
--- End diff --
this id -> the id
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
