[jira] [Commented] (BEAM-10779) Support multiple main inputs and side inputs for PTransformOverride

["Beam JIRA Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/BEAM-10779?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17211188#comment-17211188 ] 

Beam JIRA Bot commented on BEAM-10779:
--------------------------------------

This issue is assigned but has not received an update in 30 days so it has been labeled "stale-assigned". If you are still working on the issue, please give an update and remove the label. If you are no longer working on the issue, please unassign so someone else may work on it. In 7 days the issue will be automatically unassigned.

> Support multiple main inputs and side inputs for PTransformOverride
> -------------------------------------------------------------------
>
>                 Key: BEAM-10779
>                 URL: https://issues.apache.org/jira/browse/BEAM-10779
>             Project: Beam
>          Issue Type: New Feature
>          Components: sdk-py-core
>            Reporter: Yifan Mai
>            Assignee: Yifan Mai
>            Priority: P2
>              Labels: stale-assigned
>          Time Spent: 8.5h
>  Remaining Estimate: 0h
>
> Currently, {{PTransformOverride}} only supports overriding {{PTransform}} s that have a [single  input|https://github.com/apache/beam/blob/cfa448d121297398312d09c531258a72b413488b/sdks/python/apache_beam/pipeline.py#L311-L317]. It would be useful to support multiple main inputs and side inputs as well.
> Proposal:
> * Introduce a new method {{PTransformOverride.get_replacement_inputs(self, applied_ptransform)}} that can be overridden by end users. This method will be called to get the inputs for the replacement PTransform.
> * The default implementation will return a tuple of all the main inputs in order, concatenated with the all the side inputs {{PValue}} s in order.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)