On 23 October 2010 03:58, Justin Deoliveira <[email protected]> wrote: > Ciao Simone, > Thanks for the feedback. You are indeed correct in that what we are calling > a dao is more of a facade. Apologies I am not so well versed when it comes > to such terminology. > I would be fine with pushing for a more strict definition of dao in the > design, and acutally break out a dao interface for each type of object, and > rename currently what are calling a dao to a facade. While I agree this > would be a better design it seems more of a semantic thing. And at this > point I think we have bigger fish to fry, like modifying the apis to support > querying with paging, etc... the basic scalability stuff, which was also a > concern of yours ;) > So, while I think this is worth while I would like to push this off to a > future iteration of the work. However I would be fine with renaming > CatalogDAO and GeoServerDAO to CatalogFacade and GeoServerFacade (or > whatever name makes more sense) now. > I also don't think this will be very problematic to add layer. Currently the > Catalog and GeoServer interfaces are acting as facades for the dao/facades > interfaces, so there api is shielded from the rest of the code base. And in > the same way the dao/facade api would protect the catalog from multiple daos > per type hierarchy when we decide to break them out. > > 2c. > So is renaming the interfaces enough to get your +1? Or should I mark you as > +0? > -Justin
Sorry for the pathetic response time on this one, I've been struggling to get my mind around it. I do fully support the direction, so I'm a +1. ------------------------------------------------------------------------------ Nokia and AT&T present the 2010 Calling All Innovators-North America contest Create new apps & games for the Nokia N8 for consumers in U.S. and Canada $10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store http://p.sf.net/sfu/nokia-dev2dev _______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
