+1 on the backport.
On Tue, Dec 4, 2012 at 3:56 AM, Andrea Aime <[email protected]>wrote:
> On Tue, Dec 4, 2012 at 10:29 AM, Ben Caradoc-Davies <
> [email protected]> wrote:
>
>> The patch looks good to me. The forward and backward migration issues are
>> well-thought-out. However, I am not sure where this change stands with
>> respect to API stability.
>>
>> The stable branch is meant to have a stable API. Because this change adds
>> extra methods to LayerGroupInfo, if I have a third-party plugin that
>> modifies the catalog and adds a new LayerGroupInfo implementation, it will
>> be broken by this change because the third-party class does not implement
>> the new methods. See the changes required to DecoratingLayerGroupInfo, for
>> example.
>>
>> What do we consider to be the public API? Adding methods to a public
>> interface will break third-party implementations. But perhaps the catalog
>> is seen is internal to GeoServer?
>>
>
> Right, telling what public API is kind of hard in GeoServer.
> Generally speaking I think in terms of extension points that people can
> implement, plugins.
>
> However I cannot imagine one that "plugs" the catalog right now, it would
> have to be a downright fork instead of a plugin.
> I don't believe we should care for people keeping their fork of GeoServer
> here, if anything, we should push for these
> forks to emerge and join back.
>
> Cheers
> Andrea
>
> --
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
> information.
> ==
>
> Ing. Andrea Aime
> @geowolf
> Technical Lead
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax: +39 0584 1660272
> mob: +39 339 8844549
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>
>
>
> ------------------------------------------------------------------------------
> LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
> Remotely access PCs and mobile devices and provide instant support
> Improve your efficiency, and focus on delivering more value-add services
> Discover what IT Professionals Know. Rescue delivers
> http://p.sf.net/sfu/logmein_12329d2d
> _______________________________________________
> Geoserver-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>
>
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
