You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flink.apache.org by "Zhu Zhu (Jira)" <ji...@apache.org> on 2020/02/20 02:20:00 UTC
[jira] [Created] (FLINK-16180) Replacing vertexExecution in
ScheduledUnit with executionVertexID
Zhu Zhu created FLINK-16180:
-------------------------------
Summary: Replacing vertexExecution in ScheduledUnit with executionVertexID
Key: FLINK-16180
URL: https://issues.apache.org/jira/browse/FLINK-16180
Project: Flink
Issue Type: Improvement
Components: Runtime / Coordination
Affects Versions: 1.11.0
Reporter: Zhu Zhu
Assignee: Zhu Zhu
Fix For: 1.11.0
{{ScheduledUnit#vertexExecution}} is nullable but {{ProgrammedSlotProvider}} requires it to be non-null to work. This makes {{ProgrammedSlotProvider}} not able to be used by new scheduler tests since {{vertexExecution}} is never set in the new scheduler code path. It blocks us from reworking tests which are based legacy scheduling to base on the new scheduler.
I would propose to replace vertexExecution with a non-null executionVertexID.
This change would not break anything since {{vertexExecution}} is mainly used by {{ProgrammedSlotProvider}} in tests and logging in production. And the log would be broken if it is null.
This would also fix a potential bug, as reported in FLINK-16145.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)