[
https://issues.apache.org/jira/browse/MAPREDUCE-6395?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14594255#comment-14594255
]
Hudson commented on MAPREDUCE-6395:
-----------------------------------
FAILURE: Integrated in Hadoop-Mapreduce-trunk #2179 (See
[https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2179/])
MAPREDUCE-6395. Improve the commit failure messages in MRAppMaster recovery.
(Brahma Reddy Battula via gera) (gera: rev
10107243be66bae2212a2cd8575f9f5ade13fe9e)
*
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/MRAppMaster.java
* hadoop-mapreduce-project/CHANGES.txt
> Improve the commit failure messages in MRAppMaster recovery
> -----------------------------------------------------------
>
> Key: MAPREDUCE-6395
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6395
> Project: Hadoop Map/Reduce
> Issue Type: Improvement
> Components: applicationmaster
> Affects Versions: 2.7.0
> Reporter: Gera Shegalov
> Assignee: Brahma Reddy Battula
> Labels: noob
> Fix For: 2.8.0
>
> Attachments: MAPREDUCE-6395-002.patch, MAPREDUCE-6395-003.patch,
> MAPREDUCE-6395.patch
>
>
> There are typos, and pluralis majestatis (royal we) or user-including "we"
> messages that are confusing to the users:
> {code}
> if (commitSuccess) {
> shutDownMessage = "We crashed after successfully committing.
> Recovering.";
> forcedState = JobStateInternal.SUCCEEDED;
> } else if (commitFailure) {
> shutDownMessage = "We crashed after a commit failure.";
> forcedState = JobStateInternal.FAILED;
> } else {
> //The commit is still pending, commit error
> shutDownMessage = "We crashed durring a commit";
> forcedState = JobStateInternal.ERROR;
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
