[mapguide-internals] GDAL Provider update

[Traian Stanev]

Hi Jason,

Thanks for digging that one out. If the patch is not yet applied, it should be.

It makes the GDAL provider work analogously to all other providers by having feature readers increment the reference count of the connection so that the connection does not go away before the feature reader does. This was one of the reasons for the MapGuide crashes we were getting with that provider.

It also made opening a GDAL connection faster by some huge amount (factor of 10, as far as I remember).

Traian


> -----Original Message-----
> From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-
> internals-bounces at lists.osgeo.org] On Behalf Of Jason Birch
> Sent: Monday, November 22, 2010 10:24 PM
> To: MapGuide Internals Mail List
> Subject: Re: [mapguide-internals] GDAL Provider update
> 
> No kidding :)
> 
> Did you see the patches attached to this ticket?
> http://trac.osgeo.org/mapguide/ticket/462
> 
> <http://trac.osgeo.org/mapguide/attachment/ticket/462/grfp_addref.patch
> >Traian
> found and created a patch for a reference counting issue in the
> provider,
> but I don't think it's ever been applied (I think fixing it at the time
> may
> have caused a problem?)
> 
> Also, I'm still not convinced that not guarding the stylization (re
> mapguide_raster_stability.patch from Haris) call isn't risky.  I didn't
> really understand the arguments one way or the other, but it sounded
> like
> this was trading potential stability for performance.  I think this
> would be
> around here now:
> 
> http://trac.osgeo.org/mapguide/browser/trunk/MgDev/Server/src/Services/
> Mapping/MappingUtil.cpp#L718
> 
> For the record, I've had success on Windows Server 2003, 32bit,
> MapGuide
> 2.1.  Haven't tested anything newer, or x64.
> 
> Jason
> 
> On 22 November 2010 13:28, Trevor Wekel wrote:
> 
> > Hi everyone,
> >
> > I have posted some information on the current state of the GDAL
> Provider in
> > MapGuide 2.2 RC1 based on testing I have performed and community
> feedback.
> >
> >
> > http://www.otxsystems.com/community-initiatives/gdal-provider-
> initiative.html
> >
> >
> > There is certainly room for improvement.
> >
> >
> > Regards,
> > Trevor
> >
> > _______________________________________________
> > mapguide-internals mailing list
> > mapguide-internals at lists.osgeo.org
> > http://lists.osgeo.org/mailman/listinfo/mapguide-internals
> >
> >
> _______________________________________________
> mapguide-internals mailing list
> mapguide-internals at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/mapguide-internals