-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10801/
-----------------------------------------------------------

(Updated May 4, 2013, 1:09 a.m.)


Review request for oozie.


Changes
-------

OOZIEClient API change
 OozieClient.getJMSConnectionInfo(jobId) -> OozieClient.getJMSConnectionInfo()

Also new API in OozieClient
 OozieClient.getTopicName(jobId)

JMSConnectionInfo API

getTopicPrefix()
getJNDIProperties()
getTopicPattern(AppType)


Description
-------

https://issues.apache.org/jira/browse/OOZIE-1347


This addresses bug OOZIE-1347.
    https://issues.apache.org/jira/browse/OOZIE-1347


Diffs (updated)
-----

  trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1478498 
  trunk/client/src/main/java/org/apache/oozie/client/JMSConnectionInfo.java 
1478498 
  
trunk/client/src/main/java/org/apache/oozie/client/JMSConnectionInfoWrapper.java
 PRE-CREATION 
  trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java 1478498 
  
trunk/client/src/main/java/org/apache/oozie/client/event/jms/JMSHeaderConstants.java
 1478498 
  
trunk/client/src/main/java/org/apache/oozie/client/event/jms/JMSMessagingUtils.java
 1478498 
  
trunk/client/src/main/java/org/apache/oozie/client/event/jms/JSONMessageDeserializer.java
 1478498 
  
trunk/client/src/main/java/org/apache/oozie/client/event/jms/MessageDeserializer.java
 1478498 
  
trunk/client/src/main/java/org/apache/oozie/client/event/message/JobMessage.java
 1478498 
  trunk/client/src/main/java/org/apache/oozie/client/rest/JsonTags.java 1478498 
  trunk/client/src/main/java/org/apache/oozie/client/rest/JsonToBean.java 
1478498 
  trunk/client/src/main/java/org/apache/oozie/client/rest/RestConstants.java 
1478498 
  trunk/client/src/test/java/org/apache/oozie/client/rest/TestJsonToBean.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/BaseEngine.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1478498 
  
trunk/core/src/main/java/org/apache/oozie/client/rest/JMSConnectionInfoBean.java
 1476260 
  trunk/core/src/main/java/org/apache/oozie/command/JMSInfoXCommand.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/jms/DefaultConnectionContext.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/jms/DefaultJMSServerInfo.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/jms/JMSExceptionListener.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/jms/JMSJobEventListener.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/jms/JMSServerInfo.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/service/JMSAccessorService.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/service/JMSTopicService.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/servlet/BaseAdminServlet.java 
1478498 
  trunk/core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/servlet/V0AdminServlet.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/servlet/V0JobServlet.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/servlet/V1AdminServlet.java 1478498 
  trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 1478498 
  trunk/core/src/main/resources/oozie-default.xml 1478498 
  trunk/core/src/test/java/org/apache/oozie/command/TestJMSInfoXCommand.java 
1478498 
  
trunk/core/src/test/java/org/apache/oozie/jms/TestDefaultConnectionContext.java 
1478498 
  trunk/core/src/test/java/org/apache/oozie/jms/TestJMSJobEventListener.java 
1478498 
  trunk/core/src/test/java/org/apache/oozie/jms/TestJMSServerInfo.java 1478498 
  trunk/core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java 
1478498 
  trunk/core/src/test/java/org/apache/oozie/service/TestJMSTopicService.java 
1478498 
  trunk/core/src/test/java/org/apache/oozie/servlet/MockDagEngineService.java 
1478498 
  trunk/core/src/test/java/org/apache/oozie/servlet/TestV1JobServlet.java 
1478498 

Diff: https://reviews.apache.org/r/10801/diff/


Testing
-------


Thanks,

Virag Kothari

Reply via email to