[ 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