Github user mxm commented on a diff in the pull request:
https://github.com/apache/flink/pull/896#discussion_r34548050
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/accumulators/AccumulatorEvent.java
---
@@ -18,32 +18,60 @@
package org.apache.flink.runtime.accumulators;
-import java.io.IOException;
-import java.util.Map;
-
-import org.apache.flink.api.common.accumulators.Accumulator;
import org.apache.flink.api.common.JobID;
+import org.apache.flink.api.common.accumulators.Accumulator;
+import org.apache.flink.runtime.executiongraph.ExecutionAttemptID;
import org.apache.flink.runtime.util.SerializedValue;
+import java.io.IOException;
+import java.io.Serializable;
+import java.util.Map;
+
/**
- * This class encapsulates a map of accumulators for a single job. It is
used
+ * This class encapsulates a map of accumulators for a single task. It is
used
* for the transfer from TaskManagers to the JobManager and from the
JobManager
* to the Client.
*/
-public class AccumulatorEvent extends SerializedValue<Map<String,
Accumulator<?, ?>>> {
+public class AccumulatorEvent implements Serializable {
- private static final long serialVersionUID = 8965894516006882735L;
+ private static final long serialVersionUID = 42L;
- /** JobID for the target job */
private final JobID jobID;
+ private final ExecutionAttemptID executionAttemptID;
+ private final Map<AccumulatorRegistry.Metric, Accumulator<?, ?>>
flinkAccumulators;
+ private final SerializedValue<Map<String, Accumulator<?, ?>>>
userAccumulators;
- public AccumulatorEvent(JobID jobID, Map<String, Accumulator<?, ?>>
accumulators) throws IOException {
- super(accumulators);
+ public AccumulatorEvent(JobID jobID, ExecutionAttemptID
executionAttemptID,
--- End diff --
The class is used to transfer the accumulator results from the task manager
to the job manager. So it is used in both runtime components. I agree we could
give it a better name. I like `AccumulatorSnapshot`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
