-
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
Rename jdbc minPoolSize to minIdle #6347
Conversation
* Make name of configuration property clearer and make it more clear that BTM specific options only apply to XA datasource connection pools. * Revert old name `transactionmanager.narayana.jdbc.connection.minPoolSize` for `transactionmanager.narayana.jdbc.connection.minIdle`.
Quality Gate passedIssues Measures |
@@ -115,8 +115,12 @@ jdbc.required=true | |||
jdbc.datasource.default=jdbc/${instance.name.lc} | |||
|
|||
## JDBC Connection Pool properties | |||
# Minimum number of physical connections to maintain. | |||
# Deprecated, use instead `transactionmanager.jdbc.connection.minIdle` |
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.
Shouldn't we remove the deprecated option from 8.1 because we already marked it as deprecated in 7.9/8.0 and people are still going to migrate to that version still?
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.
Regardless when people migrate to newer versions, it's deprecated in this one, thus the tag should be there?
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.
Ok, we'll leave it for 8.1 like this.
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.
Question asked about deprecation.
transactionmanager.narayana.jdbc.connection.minPoolSize
fortransactionmanager.narayana.jdbc.connection.minIdle
.