Issue Details (XML | Word | Printable)

Key: NUCCORE-1209
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Unassigned
Reporter: Kaarel Kann
Votes: 0
Watchers: 0

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

Constructor cache in IdentityManagerImpl

Created: 20/May/14 10:21 AM   Updated: 21/May/14 05:09 PM   Resolved: 20/May/14 11:23 AM
Component/s: None
Affects Version/s: 4.0.0.m3
Fix Version/s: 4.0.0.m4

File Attachments: 1. Zip Archive (6 kB)

Severity: Development

 Description  « Hide
Finding constructor for Identity class is a bit expensive and can be optimized by caching them.
It can save up to 5% cpu.

Kaarel Kann added a comment - 20/May/14 10:39 AM
Attached performance test.

4.0.0-m3: 37411ms
NUCCORE-1209: 35164ms

Pull request filed:

Kaarel Kann made changes - 20/May/14 10:39 AM
Field Original Value New Value
Attachment [ 12069 ]
Andy Jefferson added a comment - 20/May/14 11:23 AM
GitHub master has an adapted form of this patch. Thx

Please follow the coding standards at and then I can just apply the patch direct for any future.

Andy Jefferson made changes - 20/May/14 11:23 AM
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 4.0.0.m4 [ 12184 ]
Resolution Fixed [ 1 ]
Andy Jefferson made changes - 21/May/14 05:09 PM
Status Resolved [ 5 ] Closed [ 6 ]