[postgis-devel] Making SFCGAL mandatory

[Paul Ramsey]

On Thu, Nov 15, 2018 at 8:20 AM Regina Obe <lr at pcorp.us> wrote:

>
> > -----Original Message-----
> > From: postgis-devel [mailto:postgis-devel-bounces at lists.osgeo.org] On
> Behalf
> > Of Sandro Santilli
> > Sent: Thursday, November 15, 2018 5:25 AM
> > To: PostGIS Development Discussion <postgis-devel at lists.osgeo.org>
> > Subject: Re: [postgis-devel] Making SFCGAL mandatory
> >
> > On Thu, Nov 15, 2018 at 01:08:54PM +0300, Darafei "Kom?pa" Praliaskouski
> > wrote:
> >
> > > >
> > > > Probably. I don't remember exactly why we chose to use GUC.
> > > > Functions in their own schema like sfcgal.ST_Intersects() could have
> > > > been a solution as well (?)
> > >
> > > Can we change that for PostGIS 3.0, so that instead of
> > > postgis.backend=sfcgal one will set search_path=postgis_sfcgal,public?
> >
> > +1
> >
> > --strk;
> > _______________________________________________
> > postgis-devel mailing list
> > postgis-devel at lists.osgeo.org
> > https://lists.osgeo.org/mailman/listinfo/postgis-devel
> [Regina Obe]
> What?  NO  -1
>
> That is a super breaking change.  We'd have to remove the postgis schema
> qualfications in SFCGAL since we wouldn't know where postgis is installed.
> We've already got issues with postgis.topology and postgis_tiger_geocoder
> because of this.
> Let's not compound the issue.
>

Agreed.

Yes, this would have be an interesting approach back when we had the
option, but sort of assumes using specific install schemas as a general
policy, which not everyone does.
Since I was complaining about it being a basically unused features, I was
this morning wondering about removing the run-time backend switch?  If we
want to keep binding to specific CGAL functions that overlap GEOS
functionality, then just naming them appropriately (CGAL_Intersects()?
ST_IntersectsCGAL()?) would hardly be the end of the world.

P.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/postgis-devel/attachments/20181115/a3b2dbd3/attachment.html>