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
49 stars 25 forks source link

Add JmsPoolConnectionFactory.close() method #39

Closed pekuz closed 2 months ago

pekuz commented 2 months ago

... or even implement AutoCloseable.

tabish121 commented 2 months ago

The factory has stop and start methods which serve the model of its intended use better than a close which implies a terminal outcome. Adding a close method adds little value over those and making it auto closeable seems like a recipe for disaster.