kurtcebe commented on PR #179: URL: https://github.com/apache/commons-dbcp/pull/179#issuecomment-1114064561
I've removed the test to check the default state (i.e. we don't disable connection MBean). It turns out that the initial states are different for different types of Datasources, for example, BasicManagedDatasource does not register connection by default, while BasicDatasource and PStmtPoolingBasicDatasource does, i.e. there is no common value to check. We still test the case when parameter is set. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
