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

Jian He edited comment on YARN-6756 at 8/28/17 9:53 PM:
--------------------------------------------------------

[~asuresh], ut added.
there's one more issue, the relaxLocality was by default  true for the old 
ContainerRequest constructors, but with the builder it is by default false
I changed it to true, looks ok ?


was (Author: jianhe):
[~asuresh], ut added.
there's one more issue, the relaxLocality was by default is true for the old 
ContainerRequest constructors, but with the builder it is by default false
I changed it to true, looks ok ?

> ContainerRequest#executionTypeRequest causes NPE
> ------------------------------------------------
>
>                 Key: YARN-6756
>                 URL: https://issues.apache.org/jira/browse/YARN-6756
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jian He
>            Assignee: Jian He
>            Priority: Critical
>         Attachments: YARN-6756.01.patch, YARN-6756.02.patch
>
>
> ContainerRequest#executionTypeRequest is initialized as null, which could 
> cause below " execTypeReq.getExecutionType" unconditionally throw NPE.
> {code}
>   ResourceRequestInfo addResourceRequest(Long allocationRequestId,
>       Priority priority, String resourceName, ExecutionTypeRequest 
> execTypeReq,
>       Resource capability, T req, boolean relaxLocality,
>       String labelExpression) {
>     ResourceRequestInfo resourceRequestInfo = get(priority, resourceName,
>         execTypeReq.getExecutionType(), capability);
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to