Jian He commented on YARN-2312:

yep, we should update the caller of getId to use getContainerId, and get this 
in for 2.6
One problem I found is that  ContainerTokenIdentifer is now still using short 
int of the containerId, which is used by NM to construct the ContainerId. 
Because of this containerId dismatch, RM doesn't recognize the containerStatus 
sent by NM at all.. 

> Marking ContainerId#getId as deprecated
> ---------------------------------------
>                 Key: YARN-2312
>                 URL: https://issues.apache.org/jira/browse/YARN-2312
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Tsuyoshi OZAWA
>            Assignee: Tsuyoshi OZAWA
> {{ContainerId#getId}} will only return partial value of containerId, only 
> sequence number of container id without epoch, after YARN-2229. We should 
> mark {{ContainerId#getId}} as deprecated and use 
> {{ContainerId#getContainerId}} instead.

This message was sent by Atlassian JIRA

Reply via email to