apache / apache/beam

The Java SDK Harness and runners-core-construction payloads should be consistent

Open
#18,702 0 comments 0 reactions 0 assignees View on GitHub
bug core harness java P3 runners
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
2d 2h
Merged PRs (30d)
205

Description

Right now the Java SDK harness expects DoFnInfo instances within the payload of all DoFns it is provided, while the pipeline translation/ParDoTranslation/etc assumes that the ParDoTranslation channel will be used to extract useful values from the serialized form (potentially by converting to a ParDoLike). As such, a Java ParDo serialized with the Java ParDoTranslation class cannot be provided in that form to the Java SDK harness. Instead, that node has to be rewritten to include a `DoFnInfo` in the payload.

This seems unreasonable, and the two should be brought in line to use the same serialized representation.

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

Contributor guide

Open the contributing guide

Research direction

Compare the Java SDK harness with runners-core-construction and the pipeline translation path, especially ParDoTranslation, DoFnInfo, and ParDoLike. Determine which serialized representation should be shared, then verify that Java ParDos can pass from translation to the harness without rewriting their payloads.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
data-engineering
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.