You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by ag...@apache.org on 2006/01/27 18:53:37 UTC
svn commit: r372928 -
/incubator/roller/trunk/src/org/roller/presentation/cache/CacheManager.java
Author: agilliland
Date: Fri Jan 27 09:53:33 2006
New Revision: 372928
URL: http://svn.apache.org/viewcvs?rev=372928&view=rev
Log:
don't invalidate a website just because of a referrer, that is not good for performance.
Modified:
incubator/roller/trunk/src/org/roller/presentation/cache/CacheManager.java
Modified: incubator/roller/trunk/src/org/roller/presentation/cache/CacheManager.java
URL: http://svn.apache.org/viewcvs/incubator/roller/trunk/src/org/roller/presentation/cache/CacheManager.java?rev=372928&r1=372927&r2=372928&view=diff
==============================================================================
--- incubator/roller/trunk/src/org/roller/presentation/cache/CacheManager.java (original)
+++ incubator/roller/trunk/src/org/roller/presentation/cache/CacheManager.java Fri Jan 27 09:53:33 2006
@@ -263,7 +263,9 @@
mLogger.debug("invalidating referer = "+referer.getId());
- lastExpiredCache.put(referer.getWebsite().getHandle(), new Date());
+ // NOTE: Invalidating an entire website for each referer is not
+ // good for our caching. This may need reevaluation later.
+ //lastExpiredCache.put(referer.getWebsite().getHandle(), new Date());
Iterator handlers = cacheHandlers.iterator();
while(handlers.hasNext()) {