> On Aug. 27, 2015, 11:58 p.m., Ying Zheng wrote: > > webapp/src/conf/oozie/conf/oozie-site.xml, line 544 > > <https://reviews.apache.org/r/37520/diff/2/?file=1042826#file1042826line544> > > > > Should it be "oozie.jms.producr.connection.properties"? typo? Same for > > the other oozie-site.xml > > Pallavi Rao wrote: > It might be a typo from oozie side.. The property works as specified. > > Ying Zheng wrote: > Did you try with external standalone ActiveMQ server? I tried to set this > property on Ambari with standalone ActiveMQ server, but couldn't get a > successful run until I changed it to > "oozie.jms.producer.connection.properties".
You are right. Fixed the typo. - Pallavi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37520/#review96799 ----------------------------------------------------------- On Oct. 8, 2015, 10:24 a.m., Pallavi Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37520/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2015, 10:24 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1231 > https://issues.apache.org/jira/browse/FALCON-1231 > > > Repository: falcon-git > > > Description > ------- > > Workflow completion is currently dependent on post processing action putting > messages (success/failure) to a JMS queue. > When failures occur on Hadoop / Oozie, the post processing action may not get > executed either. In such cases, Falcon will be blind to the status of the > workflow and re-tries will not happen. > Instead, use JMS notifications of Oozie to get job completion notifications. > > The patch is backward compatible. It will handle notifications from > post-processing and Oozie. That way users are not forced to perform any > migration steps, but, can move to Oozie notifications in a phased manner. > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/resource/InstancesResult.java > 76bb4b0 > common/src/main/java/org/apache/falcon/catalog/CatalogPartitionHandler.java > d0b09df > common/src/main/java/org/apache/falcon/entity/WorkflowNameBuilder.java > 6890594 > common/src/main/java/org/apache/falcon/metadata/MetadataMappingService.java > ef9da45 > common/src/main/java/org/apache/falcon/workflow/WorkflowExecutionArgs.java > 9456fb9 > > common/src/main/java/org/apache/falcon/workflow/WorkflowExecutionContext.java > 4454239 > > common/src/main/java/org/apache/falcon/workflow/WorkflowExecutionListener.java > 2d3a477 > > common/src/main/java/org/apache/falcon/workflow/WorkflowJobEndNotificationService.java > c4f8843 > > common/src/main/java/org/apache/falcon/workflow/engine/AbstractWorkflowEngine.java > 0b560bb > common/src/test/java/org/apache/falcon/entity/TestWorkflowNameBuilder.java > 6060731 > > common/src/test/java/org/apache/falcon/workflow/WorkflowExecutionContextTest.java > 65a057d > > common/src/test/java/org/apache/falcon/workflow/WorkflowJobEndNotificationServiceTest.java > b7df443 > messaging/pom.xml 918de63 > messaging/src/main/java/org/apache/falcon/messaging/JMSMessageConsumer.java > d3178fb > > messaging/src/test/java/org/apache/falcon/messaging/JMSMessageConsumerTest.java > 7356ee3 > oozie-el-extensions/src/main/conf/oozie-site.xml 0925b41 > oozie/src/main/java/org/apache/falcon/workflow/FalconPostProcessing.java > cff1187 > > oozie/src/main/java/org/apache/falcon/workflow/engine/OozieWorkflowEngine.java > 6660af1 > > oozie/src/test/java/org/apache/falcon/oozie/workflow/FalconPostProcessingTest.java > 9d31d17 > rerun/src/main/java/org/apache/falcon/rerun/handler/LateRerunHandler.java > 785dce8 > rerun/src/main/java/org/apache/falcon/rerun/handler/RetryHandler.java > b952bbe > webapp/src/conf/oozie/conf/oozie-site.xml 8545ef9 > > Diff: https://reviews.apache.org/r/37520/diff/ > > > Testing > ------- > > UTs added. Manual testing done. > > > Thanks, > > Pallavi Rao > >
