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

Reuven Lax commented on BEAM-6858:
----------------------------------

Inside ParDo, you will need to get the rest of the information of the 
PCollectionView, which means that somewhere a map of TupleTag -> 
PCollectionView needs to be maintained. [~kenn] do you have advice on where the 
best place to do this is?

> Support side inputs injected into a DoFn
> ----------------------------------------
>
>                 Key: BEAM-6858
>                 URL: https://issues.apache.org/jira/browse/BEAM-6858
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>            Reporter: Reuven Lax
>            Assignee: Shehzaad Nakhoda
>            Priority: Major
>
> Beam currently supports injecting main inputs into a DoFn process method. A 
> user can write the following:
> @ProcessElement public void process(@Element InputT element)
> And Beam will (using ByteBuddy code generation) inject the input element into 
> the process method.
> We would like to also support the same for side inputs. For example:
> @ProcessElement public void process(@Element InputT element, 
> @SideInput("tag1") String input1, @SideInput("tag2") Integer input2) 
> This requires the existing process-method analysis framework to capture these 
> side inputs. The ParDo code would have to verify the type of the side input 
> and include them in the list of side inputs. This would also eliminate the 
> need for the user to explicitly call withSideInputs on the ParDo.
>  



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to