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

sridhar Reddy commented on BEAM-7049:
-------------------------------------

My WIP pull request (#9358 )already contains code to handle inputs >2. I 
checked in this code on 08/23. If I can find the code which is making my 
BeamPlan different from yours then all the required code is already checked in 
to the pull request.  

I will review my code changes to see if I inadvertently changed something.

 

Thanks for your feedback.

> Merge multiple input to one BeamUnionRel
> ----------------------------------------
>
>                 Key: BEAM-7049
>                 URL: https://issues.apache.org/jira/browse/BEAM-7049
>             Project: Beam
>          Issue Type: Improvement
>          Components: dsl-sql
>            Reporter: Rui Wang
>            Assignee: sridhar Reddy
>            Priority: Major
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> BeamUnionRel assumes inputs are two and rejects more. So `a UNION b UNION c` 
> will have to be created as UNION(a, UNION(b, c)) and have two shuffles. If 
> BeamUnionRel can handle multiple shuffles, we will have only one shuffle



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to