Carlo Curino commented on YARN-2731:

Simple fix, invoking maybeInitBuilder() in setNMTokensFromPreviousAttempts() 
and moved it in setClientToAMTokenMasterKey... eyeballing the other methods 
they look ok, beside the invocation of maybeInitBuilder() in a getter: 
getClientToAMTokenMasterKey() which seems odd.

> RegisterApplicationMasterResponsePBImpl: not properly initialized builder
> -------------------------------------------------------------------------
>                 Key: YARN-2731
>                 URL: https://issues.apache.org/jira/browse/YARN-2731
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Carlo Curino
>            Assignee: Carlo Curino
>         Attachments: YARN-2731.patch
> If I am not mistaken in RegisterApplicationMasterResponsePBImpl we are 
> missing to initialize the builder in  setNMTokensFromPreviousAttempts(), and 
> we initialize the builder in the wrong place in:  setClientToAMTokenMasterKey

This message was sent by Atlassian JIRA

Reply via email to