From 4b5b1ac3d10f1a190450ad59b8be5c9568921852 Mon Sep 17 00:00:00 2001 From: Jason Lowe Date: Tue, 9 Oct 2018 13:27:03 -0500 Subject: [PATCH] MAPREDUCE-7130. Rumen crashes trying to handle MRAppMaster recovery events. Contributed by Peter Bacsko --- .../hadoop/tools/rumen/Pre21JobHistoryConstants.java | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/Pre21JobHistoryConstants.java b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/Pre21JobHistoryConstants.java index 239d666f41..8adff46fa5 100644 --- a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/Pre21JobHistoryConstants.java +++ b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/Pre21JobHistoryConstants.java @@ -44,12 +44,17 @@ static enum Keys { /** * This enum contains some of the values commonly used by history log events. * since values in history can only be strings - Values.name() is used in - * most places in history file. + * most places in history file. + * + * Note: "SUCCEEDED" is actually not a pre-0.21 value, but it might appear + * in jhist logs when the event is an unsuccessful job completion, yet, the + * overall job status is "SUCCEEDED". */ public static enum Values { - SUCCESS, FAILED, KILLED, MAP, REDUCE, CLEANUP, RUNNING, PREP, SETUP + SUCCESS, SUCCEEDED, FAILED, KILLED, MAP, REDUCE, CLEANUP, RUNNING, PREP, + SETUP } - + /** * Regex for Pre21 V1(old) jobhistory filename * i.e jt-identifier_job-id_user-name_job-name