[ https://issues.apache.org/jira/browse/QPID-3044?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13066018#comment-13066018 ]
jirapos...@reviews.apache.org commented on QPID-3044: ----------------------------------------------------- ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/441/#review1077 ----------------------------------------------------------- The apparent JBoss 'dependency' for this seems less than ideal. I don't think we should/can accept something into the Qpid code base when it hasn't even been attempted to use it with a product that is actually Apache Licence compatible. I think this really needs to be able to work fully out the box with something like Geronimo, with additional rather than sole configuration supplied for JBoss AS. It probably gets round the letter of the licence, since although the example actually imports some JBoss annotations it probably gets away with it because it doesn't look to be built by default (correct?), whereas the adapter itself either has the bits turned off that are JBoss dependent and/or uses reflection to avoid the imports. That doesn't really seem to change the fact you don't appear to be able to use all of the 'optional' features without JBoss though, which is probably still in violation of the spirit of the licence. It still appears that there is absolutely zero test coverage for this? We keep talking about trying to improve our test situation as a project, but particularly in the Java tree, and additions like this are not helpful. The code continues to violate the field naming convention we use on the Java components ( https://cwiki.apache.org/qpid/java-coding-standards.html ), although in contrast to earlier versions this diff actually has a mixture of field naming conventions, making it inconsistent as well. /trunk/qpid/java/eclipse-projects/.gitignore <https://reviews.apache.org/r/441/#comment2197> Do we need an 'empty' directory in the repository for this? We have plenty of directories in the Java tree as it is without adding redundant (for me at least) ones. Could this instead be added to the existing global .gitignore, or a new one be added in the Java directory to ignore the subdirectory and its contents for those that actually wish to create it? /trunk/qpid/java/jca/example/build.xml <https://reviews.apache.org/r/441/#comment2198> Out of date hard coded value. Can we use the existing variables to keep this up to date? /trunk/qpid/java/jca/example/build.xml <https://reviews.apache.org/r/441/#comment2199> Out of date hard coded values. (project version as before, and we also just upgraded to mina 1.1.7) /trunk/qpid/java/jca/example/src/qpid/jca/example/web/QpidTestServlet.java <https://reviews.apache.org/r/441/#comment2200> There are tab characters instead of spaces in several places within this file. - Robbie On 2011-07-14 22:39:37, Andrew Stitcher wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/441/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-07-14 22:39:37) bq. bq. bq. Review request for qpid, Gordon Sim, Andrew Kennedy, Robbie Gemmell, rajith attapattu, and Weston Price. bq. bq. bq. Summary bq. ------- bq. bq. Review for a qpid JCA resource adapter. bq. bq. So far no build infrastructure is included. bq. bq. 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). bq. bq. Any and all comments welcome. bq. bq. bq. This addresses bug QPID-3044. bq. https://issues.apache.org/jira/browse/QPID-3044 bq. bq. bq. Diffs bq. ----- bq. bq. /trunk/qpid/java/lib/geronimo-j2ee-connector_1.5_spec-2.0.0.jar UNKNOWN bq. /trunk/qpid/java/lib/geronimo-jta_1.1_spec-1.1.1.jar UNKNOWN bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/inflow/QpidMessageHandler.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/tm/JBossTransactionManagerLocator.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/resources/META-INF/ra.xml PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/inflow/QpidActivationSpec.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/admin/QpidConnectionFactoryProxy.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/admin/QpidDestinationProxy.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/admin/QpidQueue.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/admin/QpidTopic.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/inflow/QpidActivation.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/admin/AdminObjectFactory.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/admin/QpidBindingURL.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/Util.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidResourceAdapter.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAStreamMessage.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRATextMessage.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRATopicPublisher.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRATopicSubscriber.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAXAResource.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRASessionFactory.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRASessionFactoryImpl.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAQueueBrowser.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAQueueReceiver.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAQueueSender.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRASession.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAObjectMessage.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAProperties.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMetaData.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessageProducer.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessageListener.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessageConsumer.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMessage.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMapMessage.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAManagedConnectionFactory.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAManagedConnection.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAMCFProperties.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRACredential.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAException.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRALocalTransaction.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionManager.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionMetaData.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionRequestInfo.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionFactory.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRAConnectionFactoryImpl.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/QpidRABytesMessage.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/ConnectionFactoryProperties.java PRE-CREATION bq. /trunk/qpid/java/jca/src/main/java/org/apache/qpid/ra/ConnectionFactoryObjectFactory.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/web/QpidTestServlet.java PRE-CREATION bq. /trunk/qpid/java/jca/qpid-jca-ds.xml PRE-CREATION bq. /trunk/qpid/java/jca/example/conf/application.xml PRE-CREATION bq. /trunk/qpid/java/jca/example/conf/qpid-jca-ds.xml PRE-CREATION bq. /trunk/qpid/java/jca/example/conf/web.xml PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/QpidClient.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/QpidStandaloneClient.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/ejb/QpidListener.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/ejb/QpidTest.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/ejb/QpidTestBean.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/ejb/QpidTestLocal.java PRE-CREATION bq. /trunk/qpid/java/jca/example/src/qpid/jca/example/ejb/QpidTestRemote.java PRE-CREATION bq. /trunk/qpid/java/build.deps 1070497 bq. /trunk/qpid/java/build.xml 1070497 bq. /trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQQueueBrowser.java 1070497 bq. /trunk/qpid/java/client/src/main/java/org/apache/qpid/client/TopicPublisherAdapter.java 1070497 bq. /trunk/qpid/java/eclipse-projects/.gitignore PRE-CREATION bq. /trunk/qpid/java/jca/README-JBOSS.txt PRE-CREATION bq. /trunk/qpid/java/jca/README.txt PRE-CREATION bq. /trunk/qpid/java/jca/build.xml PRE-CREATION bq. /trunk/qpid/java/jca/example/.gitignore PRE-CREATION bq. /trunk/qpid/java/jca/example/README PRE-CREATION bq. /trunk/qpid/java/jca/example/build.xml PRE-CREATION bq. bq. Diff: https://reviews.apache.org/r/441/diff bq. bq. bq. Testing bq. ------- bq. bq. This code is now substantially the same as tested by Red Hat, passes the TCK. bq. bq. bq. Thanks, bq. bq. Andrew bq. bq. > Implement JCA Adapter for Java JMS client > ----------------------------------------- > > Key: QPID-3044 > URL: https://issues.apache.org/jira/browse/QPID-3044 > Project: Qpid > Issue Type: New Feature > Components: Java Client > Affects Versions: 0.8, 0.9, 0.10 > Reporter: Andrew Stitcher > Assignee: Andrew Stitcher > Labels: JCA, JMS, Java, qpid > Fix For: 0.11 > > > Currently there is no way to integrate the use of Qpid messaging into a Java > Application Server. > The solution is to create a JCA (J2EE Connector Architecture) adapter to allow > the Qpid JMS client to correctly work with the J2EE container. > This is an entirely new Java component, but may require small amounts of > change to the > JMS client code. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- Apache Qpid - AMQP Messaging Implementation Project: http://qpid.apache.org Use/Interact: mailto:dev-subscr...@qpid.apache.org