apache / apache/beam

Should MetricsContainer actually be Serializable?

Open
#19,328 0 comments 0 reactions 0 assignees View on GitHub
bug core java P3
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

Interface MetricsContainer extends Serializable but does not explain why. While it does seem like an interface where some implementations will be serialized and shipped, it does not seem like an inherent property of a MetricsContainer. In particular, Dataflow worker has some implementations that are _not_ serializable, and findbugs noticed.

Imported from Jira [BEAM-6573](https://issues.apache.org/jira/browse/BEAM-6573). Original Jira may contain additional context.
Reported by: kenn.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.