Fix for requiring oracle.jms.AQjmsFactory

reidmorrison / jruby-jms

Complete JRuby API into Java Messaging Specification (JMS)

Apache License 2.0

47 stars 21 forks source link

Fix for requiring oracle.jms.AQjmsFactory #16

Closed AlphaCluster closed 10 years ago

AlphaCluster commented 10 years ago

Added a check to make sure that AQ was being used before we load the oracle_a_q_connection factory. If this isn’t done oracle.jms.AQjmsFactory is attempted to be pulled in.

Also updated examples for HornetQ 2.4.

AlphaCluster commented 10 years ago

I don't disagree its not ideal but this was more a quick fix to resolve it till I could think of a cleaner way. Mostly this was so I could use this library in logstash.

damienclaveau commented 10 years ago

Totally agree, I am not experienced enough with JRuby to find the best solution for the first issue in oracle_a_q_connection_factory.rb. I need to make some further tests and further readings on the language :-)