apache / apache/beam

[Bug][Go SDK]: register.CombineN doesn't accept combiners with context.Context parameter

Open
#23,550 0 comments 0 reactions 0 assignees View on GitHub
bug go P2
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

### What happened?

It's valid to have combiners with a context parameter in any of the methods, for metrics and logging use, but the generic registration package doesn't tolerate it at this time.

Work arounds include continue using the static analysis code generator (starcgen), or not optimizing the combiner, or removing the context.Context issue.

Fix requires adjusting the generation template for the register package to also have acceptable extraction interfaces for Combine lifecycle methods with context parameters.

### Issue Priority

Priority: 2

### Issue Component

Component: sdk-go

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.