Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also compare across forks.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also compare across forks.
base fork: rsumbaly/azkaban
base: fc5c5747f5
...
head fork: rsumbaly/azkaban
compare: 224d334cb9
  • 2 commits
  • 1 file changed
  • 0 commit comments
  • 2 contributors
Commits on May 02, 2012
@rbpark rbpark Merge pull request #1 from rsumbaly/executor
Adding failure emails to be sent separately from success emails
4170c3e
Commits on May 09, 2012
@rsumbaly notify.emails are sent always. notify.failure.emails is sent only on …
…failures
224d334
Showing with 3 additions and 1 deletion.
  1. +3 −1 azkaban/src/java/azkaban/jobs/JobExecutorManager.java
View
4 azkaban/src/java/azkaban/jobs/JobExecutorManager.java
@@ -438,7 +438,9 @@ public void run() {
emailList = jobDescriptor.getEmailNotificationList();
final List<String> finalEmailList = emailList;
- final List<String> failedEmailList = jobDescriptor.getEmailFailureNotificationList();
+ final List<String> failedEmailList = new ArrayList<String>();
+ failedEmailList.addAll(jobDescriptor.getEmailFailureNotificationList());
+ failedEmailList.addAll(finalEmailList);
senderAddress = jobDescriptor.getSenderEmail();
final String senderEmail = senderAddress;

No commit comments for this range

Something went wrong with that request. Please try again.