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

ASF GitHub Bot logged work on BEAM-10779:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 31/Aug/20 19:47
            Start Date: 31/Aug/20 19:47
    Worklog Time Spent: 10m 
      Work Description: yifanmai commented on pull request #12655:
URL: https://github.com/apache/beam/pull/12655#issuecomment-683999886


   The remaining failing tests seem unrelated and non-spurious. How should I 
resolve this?
   
   **Portable_Python**
   > docker: Error response from daemon: Conflict. The container name 
"/license-container-python" is already in use by container 
"a52b4fd35d9c5c813b2318e1ca5d4da3c8649f2756a78890f61648f1f450c6c0". You have to 
remove (or rename) that container to be able to reuse that name.
   
   **Python**
   
   Tasks :sdks:python:test-suites:tox:py35:testPy35Cloud and 
:sdks:python:test-suites:tox:py35:testPy35Cython both fail with:
   
   > SystemError: Parent module 'setuptools' not loaded, cannot perform 
relative import


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 476784)
    Time Spent: 2h 40m  (was: 2.5h)

> 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
>          Time Spent: 2h 40m
>  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)

Reply via email to