search

messaginghub / pooled-jms

A JMS Connection pool for messaging applications supporting JMS 1.1 and 2.0 and Jakarta JMS clients
Apache License 2.0
50 stars 24 forks source link

Throw JMSException if enlistResource returns false #33

Closed graben closed 12 months ago

graben commented 1 year ago

@tabish121 : Issue sounds reasonable to better throw Exception to signal something wrong happened.

resolves #32

renebos99 commented 1 year ago

Could someone give an indication of when this will be merged an released?

graben commented 1 year ago

@gemmellr, @tabish121 : Could you plz have a look on this. Thx ;)

tabish121 commented 12 months ago

This small change doesn't account for the loaned session being lost on error and has no tests, a more complete fix was made in https://github.com/messaginghub/pooled-jms/commit/c79b2b37d270b21309c5b9da95824e1aab8dfd7c