[jira] [Commented] (FLINK-6167) Consider removing ArchivedExecutionGraph classes

["Chesnay Schepler (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/FLINK-6167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16025931#comment-16025931 ] 

Chesnay Schepler commented on FLINK-6167:
-----------------------------------------

They are actually still used, before the EG is passed to the MemoryArchivist they are being archived. This has the nice side effect of only retaining references that we actually need, independent of future changes..

> Consider removing ArchivedExecutionGraph classes
> ------------------------------------------------
>
>                 Key: FLINK-6167
>                 URL: https://issues.apache.org/jira/browse/FLINK-6167
>             Project: Flink
>          Issue Type: Improvement
>          Components: JobManager
>    Affects Versions: 1.3.0
>            Reporter: Chesnay Schepler
>            Priority: Minor
>
> The Archived* versions of the ExecutionGraph classes (ExecutionGraph, ExecutionJobVertex, ExecutionVertex, Execution) were originally intended to provide a serializable object that can be transferred to the History Server.
> The revised implementation of the history server however no longer requires them.
> As such we could either remove them, or keep them for testing purposes (instead of mocking) as they simplify the testing of the web-interface handlers quite a lot, which would however require keeping the Access* interfaces.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)