[
https://issues.apache.org/jira/browse/MAPREDUCE-7151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16655005#comment-16655005
]
Bilwa S T commented on MAPREDUCE-7151:
--------------------------------------
[~bibinchundatt] Thank you for reviewing the patch
i have updated patch.
> RMContainerAllocator#handleJobPriorityChange expects application_priority
> always
> --------------------------------------------------------------------------------
>
> Key: MAPREDUCE-7151
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-7151
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Reporter: Bibin A Chundatt
> Assignee: Bilwa S T
> Priority: Major
> Attachments: MAPREDUCE-7151-001.patch, MAPREDUCE-7151-002.patch
>
>
> As per yarn_service.proto {{AllocateResponseProto}} *application_priority* is
> *optional* field.
> But {{RMContainerAllocator#handleJobPriorityChange}} expects to have
> application priority always in response which is not mandatory.
> {code}
> private void handleJobPriorityChange(AllocateResponse response) {
> Priority priorityFromResponse = Priority.newInstance(response
> .getApplicationPriority().getPriority());
> // Update the job priority to Job directly.
> getJob().setJobPriority(priorityFromResponse);
> }
> {code}
> {code}
> message AllocateResponseProto {
> ..
> optional hadoop.common.TokenProto am_rm_token = 12;
> optional PriorityProto application_priority = 13;
> optional CollectorInfoProto collector_info = 14;
> ..
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org
