Испозую BasicDataSource в спринг контроллере.

Вопрос в том надо ли закрывать connection ? Или так это пuл не надо.   

Tony
Не знаком с этой реализаций data source, но здравый смысл подсказывает, что если connection pool там реализован, то работаешь ты не с настоящим jdbc connection, а с некоторой его оболочкой, и вызов close() не закроет реальный jdbc connection, а просто вернет его в pool. В тех реализация connection pool, с которыми я сталкивался, вызов close() был нужен именно для этого -- чтобы сообщить менеджеру пула, что connection освобожден и его можно реиспользовать. Думаю, если в дебаге посмотреть какой реальный инстанс какого класса возвращает dataSource.getConnection(), и глянуть на этот класс -- то все станет ясно.

Добавлено через 1 минуту и 14 секунд
да, забыл вывод добавить  -- закрывать надо 

Проверил logger.info(dataSource.getMaxActive()); Даже после закрития соединения maxActive показывает сколько живо = при инициализации датасоурца maxActive. Нас4ёт 4то эта залиба: jakarta dbcp.  

А maxActive должен как-то меняться? Это максимально возможное количество открытых соединений. Открыли два новых, максимально возможное число должно уменьшиться? Или увеличиться?

Согласен не подумал.Он всё время будет одинаковый. 

независимо от реализации datasource и пула соединений коннекшены из клиентского кода закрывать надо явно.


понятно, что в большинстве реализаций пулов при дефолтных настройках метод закрытия соединения не приведет к его физическому закрытию, а просто вернет соединение в пул.
если этого не делать явно, то соединение будет висеть как зАнятое, хотя и не используется, что на большой нагрузке сильно снизит масштабируемость приложения, всё тупо упрется в ограничение на количество открытых соединений, бОльшая часть которых не менее тупо простаивает )))
в некоторых реализациях пулов специально для предотвращения этой ситуации (connection leak) предусмотрены таймауты, и соединения, не используемые в течение этого времени насильно возвращаются в пул.

Добавлено через 1 минуту и 49 секунд

это детали реализации пула и datasource.
если привязывать к ним код, он получится непереносимым, что не есть гуд )

Спасибо пацаны.   Теперь буду спать спокойно, что закрывал соединения, а то му4ило меня.