[
https://issues.apache.org/jira/browse/TEZ-1711?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14188285#comment-14188285
]
Jeff Zhang commented on TEZ-1711:
---------------------------------
Attach patch.
* Just remove the cache in get getOutputSpecList
* One unit test is affected, fix it.
[~sseth], [~bikassaha] please help review.
> Don't cache outputSpecList in VertexImpl.getOutputSpecList(taskIndex)
> ---------------------------------------------------------------------
>
> Key: TEZ-1711
> URL: https://issues.apache.org/jira/browse/TEZ-1711
> Project: Apache Tez
> Issue Type: Bug
> Affects Versions: 0.5.1
> Reporter: Jeff Zhang
> Assignee: Jeff Zhang
> Attachments: TEZ-1711.patch
>
>
> It would cache the outputSpecList in its
> VertexImpl.getOutputSepcList(taskIndex), but I don't think we should cache it
> as it depends on the taskIndex, although in all the EdgeManagerPlugin
> Implementations, the value is the same no matter what the taskIndex is. But
> it has risk that if we have a new EdgeManagerPlugin that has different
> behavior. Or if this case would never happens, then just remove the taskIndex
> from the method parameter.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
