<div dir="ltr">From my understanding of the RFC, legacy functions GDALOpen() would only try to open the dataset using raster drivers and OGROpen() would only try using vector drivers, so it shouldn't take more time than in gdal 1.x.<div>
<br></div><div>It also seems that this RFC addresses the need to specify the driver(s) used to open the dataset. This effectively solves the problems raised in RFC 36, as Even mentions. In my view no need to work on RFC 36 as this should cover it (as long as the utilities are updated).</div>
<div><br></div><div>Looks good to me!</div><div><br></div><div>cheers,</div><div>Etienne</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 16, 2014 at 8:36 AM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@mines-paris.org" target="_blank">even.rouault@mines-paris.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ivan,<br>
<br>
yes indeed, I missed that one. I'll update the RFC with it.<br>
I think that the intent of RFC36 is covered by what is already proposed in RFC46<br>
with the papszAllowedDrivers of GDALOpenEx().<br>
<br>
I do not think that specifying the candidate driver(s) is that usefull in<br>
utilities since the time to launch them will be generally higher than the<br>
probing time. But that could be easily implemented later with an option if that<br>
was really needed (e.g. gdalinfo foo.tif -driver gtiff or possibly gdalinfo<br>
foo.jp2 -driver jp2ecw,jp2openjpeg).<br>
Due to the work I did in OGR drivers to avoid repeated file access by using<br>
GDALOpenInfo*, I don't think that GDAL drivers should be affected. And the OGR<br>
drivers are registered after the GDAL ones. So if the file being opened is<br>
really a GDAL dataset, the OGR drivers should have 0 impact on the opening time.<br>
I did try to benchmark a bit that, but it is difficult to come to a firm<br>
conclusion since there are many factors: hot/cold runs, whether you spawn a new<br>
process or use an existing one, etc.. And the time being measured are a few tens<br>
of milliseconds, so very sensitive to task scheduling.<br>
<span class="HOEnZb"><font color="#888888"><br>
Even<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Even,<br>
><br>
> I think that RFC 36 should be included in your list of<br>
> related RFCs and should be reconsider for adoption since the unification<br>
>  would make the probing even more expensive.<br>
><br>
> Regards,<br>
><br>
> Ivan<br>
><br>
> > From: <a href="mailto:even.rouault@mines-paris.org">even.rouault@mines-paris.org</a><br>
> > To: <a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
> > Date: Thu, 15 May 2014 20:47:32 +0200<br>
> > Subject: Re: [gdal-dev] Call for discussion on RFC 46: GDAL/OGR unification<br>
> ><br>
> > Le jeudi 08 mai 2014 00:13:22, Even Rouault a écrit :<br>
> > > Hi,<br>
> > ><br>
> > > This is a call for discussion on "RFC 46: GDAL/OGR unification"<br>
> > ><br>
> > >   <a href="http://trac.osgeo.org/gdal/wiki/rfc46_gdal_ogr_unification" target="_blank">http://trac.osgeo.org/gdal/wiki/rfc46_gdal_ogr_unification</a><br>
> > ><br>
> ><br>
> > No reaction : no interest or no time to review yet ?<br>
> > Or should I move that forward ?<br>
> > But I'd prefer if such architectural changes could be a bit reviewed...<br>
> ><br>
> > > Best regards,<br>
> > ><br>
> > > Even<br>
> ><br>
> > --<br>
> > Geospatial professional services<br>
> > <a href="http://even.rouault.free.fr/services.html" target="_blank">http://even.rouault.free.fr/services.html</a><br>
> > _______________________________________________<br>
> > gdal-dev mailing list<br>
> > <a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
> > <a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
><br>
<br>
<br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
</div></div></blockquote></div><br></div>