Refactoring JBossCache interceptor chain construction

[manik]

Ref:
http://jira.jboss.com/jira/browse/JBCACHE-338

I'm looking into JBCACHE-338 before I start looking at any more new features for 1.3 (invalidation, cache loader chaining/clustering, buddy replication) - does anyone have any issue around my moving the cache loader construction logic from TreeCache to a separate InterceptorChainBuilder class? This class could then grow to have more complexity such as interceptor configuration, the use of custom interceptor chains for 'advanced' users, etc.

Here is what I foresee:

interceptor_chain = InterceptorChainBuilder.getInstance(this).buildInterceptorChain();

to replace:

createInterceptorChain();

in TreeCache._createService().

[brian.stansberry]

+1. TreeCache has 4572 lines right now!

[anil.saldhana]

Also can you guys extract an interface so that people wishing to know what operations, TreeCache supports, do not have to go thru the ocean?

[manik]

There is a JIRA task for that - to front TreeCache with an interface (proably called 'Cache') and have this built by a CacheFactory.

http://jira.jboss.com/jira/browse/JBCACHE-57

This is only scheduled for JBossCache 2.0 though, since it is a major API change for anyone using JBossCache.

[belaban]

-1 for the following:

interface ISomething {
 void singleMethod();
}

:-)