apache / apache/beam

Disallow empty step names in ProcesBundleDescriptors

Open
#19,275 0 comments 0 reactions 0 assignees View on GitHub
fn-execution java new feature P3
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

I discovered some tests were setting empty step names, when I added  a precondition check to MetricsContainerStep map when adding java SDK metrics. Remove the code which returns the the default non-stepped unbounded MetricsContainer after ensuring step names are never passed in empty or null in tests or from a runner.

Imported from Jira [BEAM-6538](https://issues.apache.org/jira/browse/BEAM-6538). Original Jira may contain additional context.
Reported by: ajamato@google.com.

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.