Issue Details (XML | Word | Printable)

Key: NUCAPIJDO-75
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Unassigned
Reporter: Kaarel Kann
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
DataNucleus API JDO

Decrease lock contention during PersistenceManager.close()

Created: 21/May/14 08:32 AM   Updated: 29/May/14 09:01 AM   Resolved: 21/May/14 09:10 AM
Component/s: None
Affects Version/s: 4.0.0.m3
Fix Version/s: 4.0.0.m4


 Description  « Hide
Move JDOPersistenceManager.internalClose() outside of the sync block.

Improves ~7-8% https://github.com/datanucleus/tests/tree/master/jdo/performance test if used with multi-threaded configuration:

threadCount=10
count=60000

Results:
trunk: COMPLETE: getObjectById time(ms)=29148
Modifie: COMPLETE: getObjectById time(ms)=26703



Andy Jefferson added a comment - 21/May/14 09:10 AM
GitHub master has this; very minimal change unless high contention

Andy Jefferson made changes - 21/May/14 09:10 AM
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 4.0.0.m4 [ 12190 ]
Resolution Fixed [ 1 ]
Andy Jefferson added a comment - 21/May/14 09:28 AM
Test now updated with second configuration, multiple threads etc, so usable for higher contention testing

Andy Jefferson made changes - 29/May/14 09:01 AM
Status Resolved [ 5 ] Closed [ 6 ]