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

Tsuyoshi OZAWA commented on YARN-2830:
--------------------------------------

Hi [~jeagles], thanks for your reporting and contribution. I checked the source 
code of Tez Local Mode:

{code}
          ContainerId cId = ContainerId.newInstance(applicationAttemptId, 1);
{code}

I think it's more straight forward to fix as following at Tez than adding 
int-type newInstance method:

{code}
          ContainerId cId = ContainerId.newInstance(applicationAttemptId, 1L);
{code}

What do you think? cc: [~bikassaha]

> Add backwords compatible ContainerId.newInstance constructor for use within 
> Tez Local Mode
> ------------------------------------------------------------------------------------------
>
>                 Key: YARN-2830
>                 URL: https://issues.apache.org/jira/browse/YARN-2830
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>            Priority: Blocker
>         Attachments: YARN-2830-v1.patch
>
>
> YARN-2229 modified the private unstable api for constructing. Tez uses this 
> api (shouldn't, but does) for use with Tez Local Mode. This causes a 
> NoSuchMethod error when using Tez compiled against pre-2.6. Instead I propose 
> we add the backwards compatible api since overflow is not a problem in tez 
> local mode.



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

Reply via email to