Tsuyoshi OZAWA commented on YARN-2830:

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

          ContainerId cId = ContainerId.newInstance(applicationAttemptId, 1);

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

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

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

Reply via email to