ArcSDEFeatureCollection contains unneeded complexity
----------------------------------------------------

                 Key: GEOT-2178
                 URL: http://jira.codehaus.org/browse/GEOT-2178
             Project: GeoTools
          Issue Type: Improvement
          Components: data arcsde
    Affects Versions: 2.5.1
            Reporter: Gabriel Roldán
            Assignee: Gabriel Roldán
             Fix For: 2.5.2, 2.6-M0


ArcSDEFeatureCollection was developed when we the connection management forced 
it to hold a connection through its lifetime.
When switched to SessionPool and command queue, I implemented Sessions in a way 
that they can always follow the grab/use/dispose cycle in smaller execution 
units (that is, Sessions are aquired in a per transaction basis and the 
returned session takes care of actually being returned to the pool or not 
depending on whether a transaction is active).
This makes for an extremely ease of use since we don't have to think on holding 
a session nor when it is or not appropriate to close it. Yet, 
ArcSDEFEatureCollection still contains some old mess that should be removed by 
simply overriding DataFeatureCollection.close() and let the returned 
ArcSDEFEatureReader call Session.dispose() without worrying about anything else

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to