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

Siddharth Seth commented on TEZ-1360:
-------------------------------------

Looks good. Couple of minor comments.
- Javadoc for getVertexParallelism needs to be fixed. "Get the vertex 
parallelism of the vertex that is the reader for this Input."
Something like "Get the total number of tasks for the vertex to which this task 
belongs"

- Can we add a simple test to verify that the Input/Output/ProcessorContext are 
seeing the parallelism specified in the TaskSpec. 
TestLogicalIOProcessorRuntimeTask could easily be modified for this.

> Provide vertex parallelism to each vertex task
> ----------------------------------------------
>
>                 Key: TEZ-1360
>                 URL: https://issues.apache.org/jira/browse/TEZ-1360
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Johannes Zillmann
>            Assignee: Gopal V
>             Fix For: 0.5.1
>
>         Attachments: TEZ-1360.1.patch, TEZ-1360.2.patch, TEZ-1360.4.patch, 
> TEZ-1360.5.patch
>
>
> It would be good for a task to get a info about the total task count of its 
> vertex.
> With this there would be an equivalent for map-reduce' {{mapred.map.tasks}} 
> and {{mapred.reduce.tasks}} and mr-applications using this could be ported to 
> Tez more easily.



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

Reply via email to