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



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/ConnectionFactoryObjectFactory.java
<https://reviews.apache.org/r/441/#comment560>

    "QPID-CF" should be external constant 
    r.get() result not checked for null



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionFactoryImpl.java
<https://reviews.apache.org/r/441/#comment561>

    "QPID-CF" should be external constant



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionRequestInfo.java
<https://reviews.apache.org/r/441/#comment564>

    null check is redundant, since instanceof will return false for a null 
value of obj



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionRequestInfo.java
<https://reviews.apache.org/r/441/#comment565>

    hashCode does not meet Object contract with equals and should include 
clientId in calculation



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionRequestInfo.java
<https://reviews.apache.org/r/441/#comment566>

    can just use type here?



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionRequestInfo.java
<https://reviews.apache.org/r/441/#comment567>

    can just use acknowledgeMode here?



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRACredential.java
<https://reviews.apache.org/r/441/#comment568>

    do we really want to use Object's toString here?



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAManagedConnectionFactory.java
<https://reviews.apache.org/r/441/#comment569>

    null check is redundant, due to use of instanceof



/trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAManagedConnectionFactory.java
<https://reviews.apache.org/r/441/#comment570>

    null check redundant, due to instanceof


- Andrew


On 2011-02-24 15:12:05, Andrew Stitcher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/441/
> -----------------------------------------------------------
> 
> (Updated 2011-02-24 15:12:05)
> 
> 
> Review request for qpid.
> 
> 
> Summary
> -------
> 
> Review for a qpid JCA resource adapter.
> 
> So far no build infrastructure is included.
> 
> I'd also like an opinion as to whether java/jca is the appropriate name for 
> this (I'm thinking perhaps java/ra would be more usual).
> 
> Any and all comments welcome.
> 
> 
> This addresses bug QPID-3044.
>     https://issues.apache.org/jira/browse/QPID-3044
> 
> 
> Diffs
> -----
> 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionFactory.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRABytesMessage.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/ConnectionFactoryProperties.java
>  PRE-CREATION 
>   /trunk/qpid/java/build.deps 1070497 
>   /trunk/qpid/java/build.xml 1070497 
>   /trunk/qpid/java/jca/build.xml PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/ConnectionFactoryObjectFactory.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionFactoryImpl.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionManager.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionMetaData.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionRequestInfo.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRACredential.java 
> PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAException.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRALocalTransaction.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMCFProperties.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAManagedConnection.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAManagedConnectionFactory.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMapMessage.java 
> PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessage.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessageConsumer.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessageListener.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessageProducer.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMetaData.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAObjectMessage.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAProperties.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAQueueReceiver.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAQueueSender.java 
> PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRASession.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRASessionFactory.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRASessionFactoryImpl.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAStreamMessage.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRATextMessage.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRATopicPublisher.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRATopicSubscriber.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAXAResource.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidResourceAdapter.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/Util.java 
> PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/inflow/QpidActivation.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/inflow/QpidActivationSpec.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/inflow/QpidMessageHandler.java
>  PRE-CREATION 
>   
> /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/tm/JBossTransactionManagerLocator.java
>  PRE-CREATION 
>   /trunk/qpid/java/jca/src/main/resources/META-INF/ra.xml PRE-CREATION 
>   /trunk/qpid/java/lib/geronimo-j2ee-connector_1.5_spec-2.0.0.jar UNKNOWN 
>   /trunk/qpid/java/lib/geronimo-jta_1.1_spec-1.1.1.jar UNKNOWN 
> 
> Diff: https://reviews.apache.org/r/441/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andrew
> 
>

Reply via email to