ops4j / org.ops4j.pax.jms

Pax JMS provides a lightweight bridge between Java Message Service (JMS) and OSGi using Declarative Services and the whiteboard pattern
Apache License 2.0
6 stars 8 forks source link

JMS Factory for Oracle AQ [PAXJMS-19] #46

Closed grgrzybek closed 5 years ago

grgrzybek commented 5 years ago

Pravin Deshmukh created PAXJMS-19

Hello,

I have used the current code from master and have added JMS Factory for Oracle AQ. I have completed my testing. Appreciate if you could review the updates and accept them.

Thanks,
Pravin


Fixed in: 1.0.2 Votes: 0, Watches: 2

grgrzybek commented 5 years ago

Grzegorz Grzybek commented

Pravin Deshmukh I see pax-jms-oracleaq uses Import-Package: oracle.jms.* - I understand that's the role of user to ensure that some proprietary bundle is installed, right?

grgrzybek commented 5 years ago

Grzegorz Grzybek commented

Fixed here in https://github.com/ops4j/org.ops4j.pax.jms/commits/master

grgrzybek commented 5 years ago

Pravin Deshmukh commented

Grzegorz Grzybek - Yes, the import of the proprietary package is to load classes for Oracle at runtime.

Thank you for reviewing and merging the code to master branch.

grgrzybek commented 5 years ago

Grzegorz Grzybek commented

Pravin Deshmukh also, if you don't track ops4j-announcements mailing list, I've released pax-jms 1.0.2 - it's already in Maven Central: https://repo1.maven.org/maven2/org/ops4j/pax/jms/pax-jms-oracleaq/1.0.2/