zeromq / jeromq-jms

JeroMQ JMS
25 stars 13 forks source link

`ZmqConnection.close` should be a no-op #12

Closed shs96c closed 5 years ago

shs96c commented 5 years ago

javax.jms.Connection is AutoCloseable, so expected usage is in a try-with-resources block. This automatically calls close when leaving the block. The current implementation of ZmqConnection.close throws an UnsupportedOperationException. Making this a no-op (and potentially logging the problem?) would allow the expected JMS usage to work.

mjeremym commented 5 years ago

Fixed in release 3.0