Avoid holding a reference to SeRasterAttr in ArcSDERasterGridCoverage2DReader
-----------------------------------------------------------------------------

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


{{ArcSDERasterGridCoverage2DReader}} holds a {{SeRasterAttr}} as instance 
variable, but {{SeRasterAttr}} holds a reference to the {{SeConnection}} it was 
created with, and surely it uses it for some operations like {{getBands()}}, 
{{getRasterInfo}}, etc.

This is potential source of problems since the SeConnection we use is a pooled 
one, and hence it should be used in the smallest execution units possible and 
then returned to the pool, but the instance variable holding a reference to a 
connection already returned to the pool may lead to unpredictable issues.

The approach is to hold enough metadata as simple objects as to get the 
{{SeRasterAttr}} for the raster when its needed and then discard it, returning 
the SeConnection to the pool with confidence it's not going to be used without 
our concent.

-- 
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:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to