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

Zhiyuan Yang edited comment on TEZ-3230 at 8/13/16 12:30 AM:
-------------------------------------------------------------

bq. Does it mean the TezException message "Only on demand routing ..." needs to 
be changed?
Sorry but I don't understand. Why does this need to be changed?
bq. But the existing taskIdMapping can be replaced by the new 
taskIdToPartitionMapping?
Oh I got your point... You are right. Finally only the taskIdToPartitionMapping 
is used. But since there isn't really difference in terms of performance and 
code readability, I'd say let's not do this change for now.


was (Author: aplusplus):
bq. Does it mean the TezException message "Only on demand routing ..." needs to 
be changed?
Sorry but I don't understand. Why does this need to be changed?
bq. But the existing taskIdMapping can be replaced by the new 
taskIdToPartitionMapping?
Oh I got your point... You are right. Finally only the taskIdToPartitionMapping 
is used. I will do the change.

> Implement vertex manager and edge manager of cartesian product edge
> -------------------------------------------------------------------
>
>                 Key: TEZ-3230
>                 URL: https://issues.apache.org/jira/browse/TEZ-3230
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Zhiyuan Yang
>            Assignee: Zhiyuan Yang
>         Attachments: TEZ-3230.1.patch, TEZ-3230.2.patch, TEZ-3230.3.patch, 
> TEZ-3230.4.patch, TEZ-3230.WIP.1.patch, TEZ-3230.WIP.2.patch, 
> TEZ-3230.WIP.3.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to