apache / apache/beam

Make StateRequestHandler::accept interface idiomatic.

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

Description

StateRequestHandler was based on some Dataflow SDK source code that does not conform to Beam coding conventions.  In particular, the accept method takes its return value as a parameter, which is a code smell.

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

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.