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

Siddharth Seth commented on TEZ-2840:
-------------------------------------

Actually, looking at this some more. I think we're better off reverting the 
patch, since it's moving the Logging into initializeInternal - instead of 
initialize itself. Having it in initialize means that the setup of MRInput gets 
logged independent of MRInputLegacy.init being invoked.

> MRInputLegacy.init should set splitInfoViaEvents
> ------------------------------------------------
>
>                 Key: TEZ-2840
>                 URL: https://issues.apache.org/jira/browse/TEZ-2840
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>             Fix For: 0.7.1, 0.8.1
>
>         Attachments: TEZ-2840.patch
>
>
> Only one path that uses initializeInternal and thus this field actually sets 
> it.



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

Reply via email to