apache/beam

Can we deprecate the side input paths through PAssert?

Open

#19,351 opened on Jun 3, 2022

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Java (4,097 forks)batch import
P3coregood first issueimprovementjava

Repository metrics

Stars
 (7,313 stars)
PR merge metrics
 (Avg merge 6d) (243 merged PRs in 30d)

Description

PAssert has two distinct paths - one uses GBK with a single-firing trigger, and one uses side inputs. Side inputs are usually a later addition to a runner, while GBK is one of the first primitives (with a single firing it is even simple). Filing this against myself to figure out why the side input version is not deprecated, and if it can be deprecated.

Marking this as a "starter" task because finding and eliminating side input version of PAssert should be fairly easy. You might need help but can ask on dev@

Imported from Jira BEAM-6257. Original Jira may contain additional context. Reported by: kenn.

Contributor guide

Research direction
Investigate the two paths in PAssert, identify the side input version, and plan its deprecation.
Tech stack
java
Domain
backend
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
GitApache Beam basics
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70