-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Connection Validation and Pool configuration #6212
Conversation
(Cherrypick from 8a26d34)
Only works for non-XA datasources currently, for XA datasources we need to still consider what is best to do as many XA drivers already implement pooling even if it's not configured.
maxLifeTimeSeconds = appConstants.getInt("transactionmanager.narayana.jdbc.connection.maxLifeTime", maxLifeTimeSeconds); | ||
maxIdle = appConstants.getInt("transactionmanager.narayana.jdbc.connection.maxIdle", maxIdle); | ||
maxLifeTime = appConstants.getInt("transactionmanager.narayana.jdbc.connection.maxLifeTime", maxLifeTime); | ||
testQuery = appConstants.getString("transactionmanager.narayana.jdbc.connection.testQuery", testQuery); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All these extra/new features we set are valuable improvements, to my humble opinion, even though perhaps Ali's main issue was solved differently.
test/src/main/configurations/JMS/ConfigurationEsbJmsListenerSender.xml
Outdated
Show resolved
Hide resolved
Quality Gate failedFailed conditions 25.4% Coverage on New Code (required ≥ 65%) See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
* Add Connection Pool Validation (#6119) (Cherrypick from 8a26d34) * PoolingJndiDataSourceFactory shouldn't check for DataSource to be XA or not XA (#6187) * Try to determine if a datasource already does pooling (#6199) Only works for non-XA datasources currently, for XA datasources we need to still consider what is best to do as many XA drivers already implement pooling even if it's not configured. (cherry picked from commit 281bf1e)
@@ -30,6 +30,7 @@ | |||
</property> | |||
</bean> | |||
|
|||
<bean id="dataSourceFactory" class="nl.nn.adapterframework.jndi.PoolingJndiDataSourceFactory" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oei!
No description provided.