[jbosscache-dev] Consistent factory method

[Ben Wang]

Manik,

This is a minor issue raised by Brian when he is constructing the bean file for AS5. Currently, we have slight way of creating a cache instance from the factory method.

In Cache, we do:

      CacheFactory factory = new DefaultCacheFactory();
      CacheSPI tree = (CacheSPI) factory.createCache(c, false);

While in PojoCache, we do:
      cache_ = PojoCacheFactory.createInstance(configFile, false);

Nothing wrong with both approaches but maybe we should be consistent in both cache instances. And I don't mind to switch if needed. For me, the reason that I did it in the first place is I don't forsee a pluggable cache instance for PojoCache. And if needed, another Factory can be used as well. 

I thought originally with your approach, there is more control over the lifecycle methods. But now it is probably not needed there. Any other reason to stick with your approach?

Thanks,

-Ben