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
