You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Stephan Ewen (JIRA)" <ji...@apache.org> on 2016/09/28 13:20:20 UTC
[jira] [Commented] (FLINK-4701) Unprotected access to cancelables
in StreamTask
[ https://issues.apache.org/jira/browse/FLINK-4701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15529571#comment-15529571 ]
Stephan Ewen commented on FLINK-4701:
-------------------------------------
I don't understand this. The constructor does not modify the set.
> Unprotected access to cancelables in StreamTask
> -----------------------------------------------
>
> Key: FLINK-4701
> URL: https://issues.apache.org/jira/browse/FLINK-4701
> Project: Flink
> Issue Type: Bug
> Reporter: Ted Yu
> Priority: Minor
>
> In performCheckpoint():
> {code}
> AsyncCheckpointRunnable asyncCheckpointRunnable = new AsyncCheckpointRunnable(
> "checkpoint-" + checkpointId + "-" + timestamp,
> this,
> cancelables,
> chainedStateHandles,
> keyGroupsStateHandleFuture,
> checkpointId,
> bytesBufferedAlignment,
> alignmentDurationNanos,
> syncDurationMillis,
> endOfSyncPart);
> synchronized (cancelables) {
> cancelables.add(asyncCheckpointRunnable);
> }
> {code}
> Construction of AsyncCheckpointRunnable should be put under the synchronized block of cancelables.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)