> On May 15, 2014, 10:54 p.m., Robert Kanter wrote: > > core/src/main/java/org/apache/oozie/service/AbandonedCoordCheckerService.java, > > line 149 > > <https://reviews.apache.org/r/21423/diff/3/?file=581724#file581724line149> > > > > What happens if Oozie isn't configured to send emails? We should > > handle that gracefully.
I see your point. We will not send email if "TO_ADDRESS = CONF_PREFIX + "email.address"" is not configured. - Purshotam ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21423/#review43175 ----------------------------------------------------------- On May 14, 2014, 6:42 p.m., Purshotam Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21423/ > ----------------------------------------------------------- > > (Updated May 14, 2014, 6:42 p.m.) > > > Review request for oozie. > > > Repository: oozie-git > > > Description > ------- > > Report will be sent in html format, where each job is clickable and will take > you to job info. > > Coordinator id > Coordinator name User name Group Kill Status > 0000000-140513184018277-oozie-puru-C COORD-TEST test > testg Coord kill is disabled > 0000001-140513184018277-oozie-puru-C COORD-TEST test > testg Coord kill is disabled > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/CoordinatorJobBean.java 7915698 > core/src/main/java/org/apache/oozie/action/email/EmailActionExecutor.java > 332d02c > core/src/main/java/org/apache/oozie/command/wf/JobXCommand.java cafeaeb > core/src/main/java/org/apache/oozie/executor/jpa/CoordJobQueryExecutor.java > 1a6ded7 > > core/src/main/java/org/apache/oozie/service/AbandonedCoordCheckerService.java > e69de29 > > core/src/test/java/org/apache/oozie/command/coord/TestAbandonedCoordChecker.java > e69de29 > > Diff: https://reviews.apache.org/r/21423/diff/ > > > Testing > ------- > > UTC > > > Thanks, > > Purshotam Shah > >
