[ https://issues.apache.org/jira/browse/QPID-8666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813273#comment-17813273 ]
ASF GitHub Bot commented on QPID-8666: -------------------------------------- gemmellr commented on code in PR #235: URL: https://github.com/apache/qpid-broker-j/pull/235#discussion_r1474621213 ########## broker-core/src/main/java/org/apache/qpid/server/store/BrokerStoreUpgraderAndRecoverer.java: ########## @@ -805,11 +805,10 @@ private static class VirtualHostEntryUpgrader addAttributeTransformer("jdbcBytesForBlob", addContextVar("qpid.jdbcstore.useBytesForBlob")). addAttributeTransformer("jdbcBlobType", addContextVar("qpid.jdbcstore.blobType")). addAttributeTransformer("jdbcVarbinaryType", addContextVar("qpid.jdbcstore.varBinaryType")). - addAttributeTransformer("partitionCount", addContextVar("qpid.jdbcstore.bonecp.partitionCount")). - addAttributeTransformer("maxConnectionsPerPartition", - addContextVar("qpid.jdbcstore.bonecp.maxConnectionsPerPartition")). - addAttributeTransformer("minConnectionsPerPartition", - addContextVar("qpid.jdbcstore.bonecp.minConnectionsPerPartition")), + addAttributeTransformer("maximumPoolSize", + addContextVar("qpid.jdbcstore.hikaricp.maximumPoolSize")). + addAttributeTransformer("minimumIdle", + addContextVar("qpid.jdbcstore.hikaricp.minimumIdle")), Review Comment: I'm not 100% sure, its a long time since I really contributed to the broker and things changed since then. Originally when we added that, I believe it was just its own independent config version. From broker-core/src/main/java/org/apache/qpid/server/model/BrokerModel.java it looks like around the 6.0.0 release (when the release version was bumped as everything became an independent component, rather than the big 'qpid release' with everything) that the model version was also jumped to bring it into 'major 6, minor 0' alignment, and since then it seems like it has consistently had changes that mean the major matched the broker release major version. It looks like maybe the minors too. Perhaps digging more at the changes that have been made historically will make it clearer for you what would be best, but using either 9.1 (next minor) or 9.2 (matching minor) seems fair if the release version is 9.2.0. > [Broker-J] Broker plugin jdbc-provider-bone replacement > ------------------------------------------------------- > > Key: QPID-8666 > URL: https://issues.apache.org/jira/browse/QPID-8666 > Project: Qpid > Issue Type: Improvement > Components: Broker-J > Affects Versions: qpid-java-broker-9.1.0 > Reporter: Daniil Kirilyuk > Priority: Major > Fix For: qpid-java-broker-9.2.0 > > > Broker plugin jdbc-provider-bone relies on JDBC connection pooling library > bonecp, which is no longer developed or updated. The [github project > page|https://github.com/wwadge/bonecp] states: > "BoneCP is a Java JDBC connection pool implementation that is tuned for high > performance by minimizing lock contention to give greater throughput for your > applications. It beats older connection pools such as C3P0 and DBCP but > should now be considered deprecated in favour of HikariCP". > Plugin jdbc-provider-bone should be replaced with the new one > jdbc-provider-hikaricp based on > [HikariCP|https://github.com/brettwooldridge/HikariCP] library. Both > libraries use Apache-2.0 license. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org