Issue Details (XML | Word | Printable)

Key: NUCCORE-773
Type: Task Task
Status: Closed Closed
Resolution: Won't Fix
Priority: Trivial Trivial
Assignee: Andy Jefferson
Reporter: Andy Jefferson
Votes: 0
Watchers: 0

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

Only update L1 cache if id not yet present

Created: 27/Oct/11 12:44 PM   Updated: 07/Nov/11 02:57 PM   Resolved: 28/Oct/11 11:12 AM
Component/s: Cache
Affects Version/s: None
Fix Version/s: 3.0.3

 Description  « Hide
Currently we call cache.put() whenever an object needs to be in the L1 cache. Obviously the L1 cache contains the StateManager keyed by the id, and so any field updates are automatically accessible from the L1 cache.

We also have to follow the other key rule that any ObjectManager (PM/EM) will only have a single object for a particular id, and so not updating the L1 cache with some other object will provide a better safeguard.

Andy Jefferson added a comment - 27/Oct/11 12:55 PM
SVN trunk clears this up

Andy Jefferson made changes - 27/Oct/11 12:55 PM
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Assignee Andy Jefferson [ andy ]
Resolution Fixed [ 1 ]
Andy Jefferson made changes - 28/Oct/11 11:11 AM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Andy Jefferson added a comment - 28/Oct/11 11:12 AM
JPA TCK had an issue with this idea, and TODO added to investigate it (later)

Andy Jefferson made changes - 28/Oct/11 11:12 AM
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Won't Fix [ 2 ]
Andy Jefferson made changes - 07/Nov/11 02:57 PM
Status Resolved [ 5 ] Closed [ 6 ]