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) 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 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) A couple other insights for another day: The class that should be called "RegistryFactoryFinder" is really only for the registry module: - It is not aimed for users! (It is for the registry module's own use) - If a module like unsupported/geometry needed to create content it should make its own GeometryFactoryFinder that gathers together the resources it needs (reuse of FactoryFinders is bad) We don't have a context: - FactoryRegistry "multiplexes" according to Hints; it is the only thing that stores state and it stores based on Hints. The FactoryFinder implementations do not store state themselves and consist only of static methods all of which take Hints Cheers, Jody ------------------------------------------------------------------------- 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
