> 
> >Jody, looks like this many have been introduced with your new wkt support 
> >in oracle?  Hans, did this just show up relatively recently, working with 
> >the corrected jars for multipolygon stuff?
> >
> doh! One more patch for 2.1.x - my bad. I wonder if we can wrap 
> ConnectionPool for test cases and fail
> when connections are not returned?
That would be good.

> 
> >that's needed is a close statement on the new determineCRS(int srid ) 
> >method in OracleDataStore.  Jody, I know you're swamped, any chance you 
> >could get it in?  All you need to do I'm 90% sure is add a 
> >finally  {
> >            JDBCUtils.close(conn, Transaction.AUTO_COMMIT, null);            
> >        }mave
> >  
> >
> You are correct - I will put the change in.
cool.

> 
> >to your determineCRS(int srid) method.  I'm sure Hans would be happy to 
> >test if you didn't want to run through the whole build process.  Or Hans, 
> >if you have a programmer who can compile geotools, you should just need to 
> >add this one line.  And we need to get Marc commit priveleges so he can do 
> >these fixes.
> >  
> >
> I am still waiting on nominating him, waiting for 1) a test case 2) 
> evidence of reading the geotools developers guide and 3) running Jalopy 
> - but his understanding of the OracleDataStore is getting better all the 
> time :-)
Yeah, I agree.

> 
> As for time, I have applied the patch and run the tests - but I only 
> will be able attach the jar (55k).
Should be sufficient, Hans, let us know if it works.

> On a related note I may have to ask you to release 2.1.1, depending on 
> when geoserver 1.3.0 actually goes out. Dave is apparently tagging 
> GeoTools for the next GeoServer RC - I had hoped to release a GeoTools 
> RC at the same time (so there were no surprises for the 2.1.1 release).
'you' being me personally?  Not likely.  'You' being TOPP, most likely we 
could do that.

c

> 
> Jody
> 

-- 



-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to