azkaban-aplcache

Fix email bug: wrong flow status for past executions (#1882)

8/3/2018 2:29:23 PM

Juho Autio

Commit: 25b41f4

Tree: 1d81d0a

Parents: 78f30ac

Changes

azkaban-common/src/main/java/azkaban/executor/mail/DefaultMailCreator.java 2(+1 -1)

azkaban-common/src/test/resources/azkaban/executor/mail/errorEmail.html 4(+2 -2)

Details

azkaban-common/src/main/java/azkaban/executor/mail/DefaultMailCreator.java 2(+1 -1)

diff --git a/azkaban-common/src/main/java/azkaban/executor/mail/DefaultMailCreator.java b/azkaban-common/src/main/java/azkaban/executor/mail/DefaultMailCreator.java
index a6483a9..9ddc488 100644
--- a/azkaban-common/src/main/java/azkaban/executor/mail/DefaultMailCreator.java
+++ b/azkaban-common/src/main/java/azkaban/executor/mail/DefaultMailCreator.java
@@ -205,7 +205,7 @@ public class DefaultMailCreator implements MailCreator {
             + convertMSToString(executableFlow.getStartTime()) + "</td></tr>");
         message.println("<tr><td>End Time</td><td>"
             + convertMSToString(executableFlow.getEndTime()) + "</td></tr>");
-        message.println("<tr><td>Status</td><td>" + flow.getStatus() + "</td></tr>");
+        message.println("<tr><td>Status</td><td>" + executableFlow.getStatus() + "</td></tr>");
         message.println("</table>");
       }

azkaban-common/src/test/resources/azkaban/executor/mail/errorEmail.html 4(+2 -2)

diff --git a/azkaban-common/src/test/resources/azkaban/executor/mail/errorEmail.html b/azkaban-common/src/test/resources/azkaban/executor/mail/errorEmail.html
index 9321eea..f457fdd 100644
--- a/azkaban-common/src/test/resources/azkaban/executor/mail/errorEmail.html
+++ b/azkaban-common/src/test/resources/azkaban/executor/mail/errorEmail.html
@@ -56,6 +56,6 @@
   </tr>
   <tr>
     <td>Status</td>
-    <td>FAILED</td>
+    <td>SUCCEEDED</td>
   </tr>
-</table>
\ No newline at end of file
+</table>