Hi Andrea: It has been a while since you first asked about this; my feedback is to use Name (rather than a String) for consistency with everything else; and to allow a bit of separation.
I would not worry about bridging the new design with the old; so far we have a udig community module and a geoserver community module to update. I would rather update both then have two concepts around of how to bundle up functionality. Distraction: I seem to recall a version being listed at some point; Simone had the need to hot deploy implementations; which trickled down into visioning different implementations of the same functionality. Jody On Sat, Jul 25, 2009 at 4:08 AM, Andrea Aime<[email protected]> wrote: > Hi, > as you know I've played a bit with hooking up Sextante with > the GeoTools process API and stumbled into issues with > the SPI and the one process per factory design: basically > SPI pretends to register a class just once, so if you reuse > the same factory over and over to register many Sextante > processes in turn (adding a factory iterator in the mix), > you end up getting only the first registered. > > This triggered the need for a process factory that can > generate multiple processes instead. > The attached patch modifies the module to follow that design: > - the factory has a Set<String> getNames() method, all other > methods accept a "name" parameter to identify the process > that will need to be created or queried about its metadata > - AbstractFactory has been replaced by SingleProcessFactory, > another abstract class that bridges the new design with the > old and allows to easily implement a factory returning a single > process > - the whole module has been updated accordingly, tests do pass > - a simple test in GeoServer WPS returns all of the Sextante > processes in the WPS capabilities (who-hoo!) > > So, please review and provide feedback on the proposed > changes. > > Cheers > Andrea > > -- > Andrea Aime > OpenGeo - http://opengeo.org > Expert service straight from the developers. > ------------------------------------------------------------------------------ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
