[
https://issues.apache.org/jira/browse/FLINK-5628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15837892#comment-15837892
]
ASF GitHub Bot commented on FLINK-5628:
---------------------------------------
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/3215#discussion_r97799282
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/CheckpointStatsTracker.java
---
@@ -85,6 +79,9 @@
/** History of checkpoints. */
private final CheckpointStatsHistory history;
+ /** The job vertices taking part in the checkpoints. */
+ private final transient List<ExecutionJobVertex> jobVertices;
--- End diff --
doesn't have to be transient anymore.
> CheckpointStatsTracker implements Serializable but isn't
> --------------------------------------------------------
>
> Key: FLINK-5628
> URL: https://issues.apache.org/jira/browse/FLINK-5628
> Project: Flink
> Issue Type: Bug
> Components: Local Runtime
> Affects Versions: 1.2.0, 1.3.0
> Reporter: Chesnay Schepler
> Assignee: Ufuk Celebi
> Priority: Blocker
> Fix For: 1.3.0
>
>
> The CheckpointStatsTracker implements the Serializable interface, even though
> it no longer is serializable as it contains a List<ExecutionJobVertex>.
> This was introduced in 579bc96446d598a2cfe8237b4ebd62d8c9df3483 which
> reworked the checkpoint stats tracking,
> This does not affect 1.2 or 1.3 in any way (since these objects aren't
> serialized there), but it blocks the implementation of the HistoryServer.
> The AccessExecution*/ArchivedExecution* classes, which are supposed to be a
> serializable form of the ExecutionGraph, are thus broken as they also make
> use of the CheckpointStatsTracker.
> (Note: * = Graph/ExecutionJobVertex/ExecutionVertex/Execution) .
> This wasn't catched in tests since no ExecutionJobVertices were given to the
> CheckpointStatsTracker in ArchivedExecutionGraphTest#setExecutionGraph.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
