[
https://issues.apache.org/jira/browse/TEZ-1065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14082543#comment-14082543
]
Jeff Zhang commented on TEZ-1065:
---------------------------------
Attach the patch. [~bikassaha] Please help review it.
Changes in patch
* Change DAGImpl.vertexMap from HashMap to LinkedHashMap
* Change vertexProgress in DAGStatus from HashMap to LinkedHashMap
* Add function test. Since this change involves both client side (DAGStatus)
and server side (DAGImpl), not easy to do unit test using mockito. So here I
create a new function test in tez-tests to verify the vertex order.
> DAGStatus.getVertexStatus and other vertex related API's should maintain
> vertex order
> -------------------------------------------------------------------------------------
>
> Key: TEZ-1065
> URL: https://issues.apache.org/jira/browse/TEZ-1065
> Project: Apache Tez
> Issue Type: Bug
> Affects Versions: 0.5.0
> Reporter: Bikas Saha
> Assignee: Jeff Zhang
> Priority: Blocker
> Labels: newbie
> Attachments: TEZ-1065.1.patch, Tez-1065-2.patch, Tez-1065.patch
>
>
> They should maintain the incoming vertex order. In VertexProgress e.g. lets
> use LinkedHashMap instead of HashMap.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
