Issue Details (XML | Word | Printable)

Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Daniel Baldes
Votes: 0
Watchers: 0

If you were logged in you would be able to see more operations.
DataNucleus Store RDBMS

ConnectionPool properties not marked as datastore properties

Created: 29/Jun/11 02:47 PM   Updated: 01/Aug/11 05:13 PM   Resolved: 29/Jun/11 08:37 PM
Component/s: None
Affects Version/s: 3.0.0.m5
Fix Version/s: 3.0.0.m6

File Attachments: 1. Text File datastore-pluginxml.patch (3 kB)

 Description  « Hide
Connection pool properties such as datanucleus.connectionPool.maxPoolSize are not marked 'datastore="true"' in datanucleus-store-rdbms' plugin.xml, therefore they do not end up in the StoreManager configuration and in turn they are not recognized by BoneCPDataSourceFactory, for instance.

Therefore, connection pool properties are eventually ignored.

I don't know whether the bug is that these properties are not marked with datastore="true", or if BoneCPDataSourceFactory should access those properties in another way, though.

Sort Order: Ascending order - Click to sort in descending order
Daniel Baldes added a comment - 29/Jun/11 02:49 PM
The attached patch for datanucleus-rdbm-store's trunk adds datastore="true" to all connection pool properties of the org.datanucleus.persistence_properties extension point. This works for me.

Andy Jefferson added a comment - 29/Jun/11 08:37 PM
Implementing the getXXXProperty methods performs the same yet also handles any other props accidentally left on the PersistenceConfiguration; that and your patch in SVN trunk. Thx