On 13-Mar-07, at 11:21 AM, Jody Garnett wrote: > Hi guys, I posted a productive IRC chat with martin on the wiki: > - > http://docs.codehaus.org/display/GEOTOOLS/2007/03/12/FactoryFinder > +IRC+Breakout > > Here is where I ended up: > - FactoryRegistry is *the* GeoTools singleton (this is what Jesse > needs > to add FactoryIterators to in order to make GeoTools work in Eclipse - > or in the next version of OSGi version of Spring) > - JNDI is *the other* GeoTools singleton (this is something GeoServer > would need to set prior to use)
From what I understand they aren't currently singletons but we want them to be isn't that more correct? > > Both these classes have some required "set up" prior to us using > GeoTools for the first time. > > Here is where we are at with FactoryFinder: > - CommonFactoryFinder currently maintains its own FactoryRegistry > singleton > - FactoryFinder (really is a ReferencingFactoryFinder) also has its > own > FactoryRegistry singleton FactoryFInders are currently the singletons and each one contains a FactoryRegistry. > > So here is what we need to do: > - We need to make sure all the FactoryFinders use the same > FactoryRegistry > - Set up the common FactoryRegistry once (that is add the > FactoryIterators) > - We need to allow an application to set the default JNDI context for > the library (in case two applications use GeoTools we cannot have them > trip over each other) I agree with those points. Is there already an existing SPI FactoryIteratorProvider? Or do we need to factory that out into its own class. So my question is where to go from here. It seems you want to create two main singletons: - FactoryRegistry - JNDI Perhaps we can create a class ala eclipse's Platform object which provides access to the singletons of the system. Maybe we can call it Geotools. Also who is slated to do this work you? Jesse ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
