Issue Details (XML | Word | Printable)

Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Andy Jefferson
Votes: 0
Watchers: 0

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?).

Sort Order: Ascending order - Click to sort in descending order
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