----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9622/ -----------------------------------------------------------
(Updated April 12, 2013, 7:38 p.m.) Review request for oozie. Changes ------- Addressed Mohammad's and Mona's comments Also removing 'Cannot access properties file' from JMSMessagingUtils as it seemed like a warning. Description ------- Description at https://issues.apache.org/jira/browse/OOZIE-1234 OOZIE-1209 generates events and handles them by calling appropriate listeners. This patch provides JMS implementation of those listeners. Also, the messages are serialized using JSON and there is a deserializer to construct the Java object back from json. This addresses bug OOZIE-1234. https://issues.apache.org/jira/browse/OOZIE-1234 Diffs (updated) ----- trunk/client/pom.xml 1467063 trunk/client/src/main/java/org/apache/oozie/client/event/jms/JMSHeaderConstants.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/jms/JMSMessagingUtils.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/jms/JSONMessageDeserializer.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/jms/MessageDeserializer.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/message/CoordinatorActionMessage.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/message/EventMessage.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/message/JobMessage.java PRE-CREATION trunk/client/src/main/java/org/apache/oozie/client/event/message/WorkflowJobMessage.java PRE-CREATION trunk/core/pom.xml 1467063 trunk/core/src/main/java/org/apache/oozie/event/CoordinatorActionEvent.java 1467063 trunk/core/src/main/java/org/apache/oozie/event/listener/JobEventListener.java 1467063 trunk/core/src/main/java/org/apache/oozie/event/messaging/JSONMessageSerializer.java PRE-CREATION trunk/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java PRE-CREATION trunk/core/src/main/java/org/apache/oozie/event/messaging/MessageSerializer.java PRE-CREATION trunk/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java 1467063 trunk/core/src/main/java/org/apache/oozie/jms/DefaultConnectionContext.java 1467063 trunk/core/src/main/java/org/apache/oozie/jms/JMSExceptionListener.java 1467063 trunk/core/src/main/java/org/apache/oozie/jms/JMSJobEventListener.java PRE-CREATION trunk/core/src/main/java/org/apache/oozie/service/EventHandlerService.java 1467063 trunk/core/src/test/java/org/apache/oozie/jms/TestDefaultConnectionContext.java PRE-CREATION trunk/core/src/test/java/org/apache/oozie/jms/TestJMSJobEventListener.java PRE-CREATION trunk/core/src/test/java/org/apache/oozie/service/TestEventHandlerService.java 1467063 trunk/core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java 1467063 trunk/pom.xml 1467063 Diff: https://reviews.apache.org/r/9622/diff/ Testing ------- Unit test cases added. Test case for JMS connection failure pending. End to end test pending Thanks, Virag Kothari