On Mon, 2010-08-16 at 16:36 -0400, Moore, Jonathan wrote: > Just an update here. I've done some scratch refactoring, and see how to > do this now. I'm not quite done but am perhaps halfway there. > > The main issue here is that there is a "hidden class" inside > CachingHttpClient that handles resource management and storage of > entries in an HttpCache; by extracting this we can simplify > CachingHttpClient and reduce its responsibilities. > > Where I'm going: > > The current HttpCache interface will be renamed to something like > CacheStorage; this is really just about storing cache entries. > > There will be a new interface for HttpCache that is a little higher > level, incorporating some of the code (including the resource management > code) that is currently in CachingHttpClient. There will be a primary > implementation of this that will take as constructor dependencies a > ResourceFactory and a CacheStorage object. > > This should simultaneously simplify CachingHttpClient and nicely > encapsulate resource management and CacheEntry storage together behind > the HttpCache interface. > > Jon >
Sounds like a good plan. Oleg --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For additional commands, e-mail: dev-h...@hc.apache.org