Issue Details (XML | Word | Printable)

Key: NUCSPATIAL-26
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Andy Jefferson
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
DataNucleus Types : Geospatial

SpatialHelper is badly designed and user should pass in PM(/EM)

Created: 26/Dec/12 03:31 PM   Updated: 22/Nov/13 11:14 AM   Resolved: 22/Nov/13 11:14 AM
Component/s: None
Affects Version/s: None
Fix Version/s: 3.2.4


 Description  « Hide
The current SpatialHelper (and derivatives) takes in a PMF. On each method call it creates a PM but doesn't close it afterwards. Would be much better to pass in the PM/EM (or maybe better the ExecutionContext so it doesn't impose deps?).

Andy Jefferson added a comment - 22/Nov/13 11:14 AM
SVN trunk removes all methods that create a new PM without closing it. If someone was using them they should swap to the method passing in the PM

Andy Jefferson made changes - 22/Nov/13 11:14 AM
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 3.2.4 [ 12053 ]
Resolution Fixed [ 1 ]
Andy Jefferson made changes - 22/Nov/13 11:14 AM
Status Resolved [ 5 ] Closed [ 6 ]