In reviewing docs for the user guide I have looked over the various factory finders currently in play and found some duplication.
CommonFactoryFinder has a register with FileDataStore (which FileDataStoreFinder uses which is nice). However it would be better to do the same thing as DataStoreFinder, and go through the single registery of all DataAccessFactory and do determine what is available without dupication? The second inconsistency is between CommonDataStoreFactory holding a registery of FeatureLockFactory. FeatureLockFactory is another case like DefaultQuery where we have over engineered things. There is only one implementation of FeatureLock, and we have yet to have an application need to configure GeoTools to make use of its own locking mechanism. This idea was put in place for J2EE apps so they could use their current session to control GeoTools. So what is a good thing to do here? - Use DataStoreFinder as a guide for fixing up FileDataStoreFinder, common factory finder can make use of the same fix. - deprecate out the FeatureLockFactory, and turn FeatureLock into a class. This probably should of been done as part of DefaultQuery since they are part of the same solution; it is just I use them so little that I did not think of it. Ideally I would like to see these changes made before 2.7.0 (so the CommonFactoryFinder methods can be correctly deprecated). I also note we had a discussion about making 2.7.0 recently; and did not do our traditional check for deprecation that can now be removed etc. Do we need to set up a work part on this one? Jody
------------------------------------------------------------------------------ Free Software Download: Index, Search & Analyze Logs and other IT data in Real-Time with Splunk. Collect, index and harness all the fast moving IT data generated by your applications, servers and devices whether physical, virtual or in the cloud. Deliver compliance at lower cost and gain new business insights. http://p.sf.net/sfu/splunk-dev2dev
_______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
