Re: [OCM] ocm cache

2008-01-16 Thread Christophe Lombart
Hi Paddy,

On Jan 15, 2008 10:29 PM, Padraic Hannon [EMAIL PROTECTED] wrote:

 There is a simple OCM cache which I would like to remove or at least
 make optional. Currently there is already a lot of caching going on
 within jackrabbit and the current cache implementation for OCM is
 extremely simplistic.


it is a small fix for http://issues.apache.org/jira/browse/JCR-918.
This is really a tempory solution to fix this issue.


 For example, if a node changes under the OCM it
 has no knowledge of this and the cache is not flushed. Until there is
 robust support for observation and a more plug-able architecture the
 OCM cache seems like it is more dangerous than helpful.



 WDYT?


Agree but the current solution is only used per retrieve request in order to
solve the infinitive loop issue.
We have to review this current/tempory solution to have a more robust
cache support.
I will review your patch today.

Thanks,
Christophe





 paddy




[OCM] ocm cache

2008-01-15 Thread Padraic Hannon
There is a simple OCM cache which I would like to remove or at least  
make optional. Currently there is already a lot of caching going on  
within jackrabbit and the current cache implementation for OCM is  
extremely simplistic. For example, if a node changes under the OCM it  
has no knowledge of this and the cache is not flushed. Until there is  
robust support for observation and a more plug-able architecture the  
OCM cache seems like it is more dangerous than helpful.


WDYT?

paddy