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

Bikas Saha commented on TEZ-1172:
---------------------------------

Would be useful to have the vertex name etc in the RootInputVertexManager 
exception.

configuredInputName is not set if it was null. so in the distributor case, 
where numTasks>1 without a configure event, multiple distributors may end up 
not failing this check because configuredInputName remains null?
{code}+        Preconditions.checkState(
+            configuredInputName == null || 
configuredInputName.equals(inputName),
+            "RootInputVertexManager cannot configure multiple inputs. Use a 
custom VertexManager");{code}

Rests looks good. +1.


> Allow multiple Root Input initializers to be specified per Vertex
> -----------------------------------------------------------------
>
>                 Key: TEZ-1172
>                 URL: https://issues.apache.org/jira/browse/TEZ-1172
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: TEZ-1172.1.txt, TEZ-1172.2.txt, TEZ-1172.3.txt, 
> TEZ-1172.4.txt
>
>
> The default VertexManager in Tez for this handles only 1 RootInput - since it 
> doesn't know how to combine multiple different Inputs. However, if a custom 
> vertex manager plugin is specified by users - it should be possible to 
> specify multiple RootInputs.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to