You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Gary Yao (Jira)" <ji...@apache.org> on 2019/11/28 20:47:00 UTC
[jira] [Comment Edited] (FLINK-14859) Avoid leaking unassigned Slot
in DefaultScheduler when Deployment is outdated
[ https://issues.apache.org/jira/browse/FLINK-14859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16978581#comment-16978581 ]
Gary Yao edited comment on FLINK-14859 at 11/28/19 8:46 PM:
------------------------------------------------------------
master:
beba9dc0a48ca2256d5b871ef96885811273bf0e
30447531f78d9feac12630e5c966ac9f81681381
was (Author: gjy):
master: beba9dc0a48ca2256d5b871ef96885811273bf0e
> Avoid leaking unassigned Slot in DefaultScheduler when Deployment is outdated
> -----------------------------------------------------------------------------
>
> Key: FLINK-14859
> URL: https://issues.apache.org/jira/browse/FLINK-14859
> Project: Flink
> Issue Type: Sub-task
> Components: Runtime / Coordination
> Affects Versions: 1.10.0
> Reporter: Gary Yao
> Assignee: Gary Yao
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.10.0
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> In {{DefaultScheduler#assignResourceOrHandleError()}}, if the deployment is outdated, we should release the possibly acquired {{LogicalSlot}} so that we do not leak resources.
> Below is an example to illustrate how slot leak is currently possible:
> # Vertices A1, A2, A3 are scheduled in a batch.
> # A2 acquires a slot. A1, A3 do not.
> # A1 fails due to slot allocation timeout and triggers failover ({{DefaultScheduler#cancelTasksAsync}})
> # A2 is canceled first and its returned slot is assigned to A3, which triggers {{DefaultScheduler#assignResourceOrHandleError}} of A3.
> However, A3 is not canceled yet but it is outdated because {{executionVertexVersioner#recordVertexModifications}} was already invoked
--
This message was sent by Atlassian Jira
(v8.3.4#803005)