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

Siddharth Seth commented on TEZ-3303:
-------------------------------------

Very minor: can we make this an "else if (proto.hasDetailedPartitionStats)". 
One of the two stats is populated; however this should not double count if both 
were populated.

Thanks [~ozawa] for the patch and [~mingma] for the review. Will commit after 
this change.

> Have ShuffleVertexManager consume more precise partition stats
> --------------------------------------------------------------
>
>                 Key: TEZ-3303
>                 URL: https://issues.apache.org/jira/browse/TEZ-3303
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Ming Ma
>            Assignee: Tsuyoshi Ozawa
>         Attachments: TEZ-3303.001.patch, TEZ-3303.002.patch, 
> TEZ-3303.002.patch, TEZ-3303.003.patch
>
>
> TEZ-3216 adds the support for more precise partition stats. 
> ShuffleVertexManager should be updated to consume the more precise partition 
> stats.



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

Reply via email to