Author: chetanm
Date: Fri Nov 15 06:50:16 2013
New Revision: 1542184
URL: http://svn.apache.org/r1542184
Log:
OAK-1156 - Improve the document cache invalidation logic to selectivly
invalidate doc
Changes as per review comments - Removing the TODO around lock handling as no
locks are
required in this access
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/mongomk/CacheInvalidator.java
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/mongomk/CacheInvalidator.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/mongomk/CacheInvalidator.java?rev=1542184&r1=1542183&r2=1542184&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/mongomk/CacheInvalidator.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/mongomk/CacheInvalidator.java
Fri Nov 15 06:50:16 2013
@@ -361,9 +361,6 @@ abstract class CacheInvalidator {
}
private void markUptodate(String key, long time, NodeDocument
uptodateRoot) {
- //TODO Should this be done under lock
-// Lock lock = getAndLock(key);
-// try {
NodeDocument doc = documentStore.getIfCached(Collection.NODES,
key);
if (doc == null) {
@@ -384,9 +381,6 @@ abstract class CacheInvalidator {
|| doc.getLastCheckTime() ==
uptodateRoot.getLastCheckTime()) {
doc.markUptodate(time);
}
-// } finally {
-// lock.unlock();
-// }
}
private void buildPath(StringBuilder sb) {
