Scarab commit: svn commit: r10448 - trunk/src: conf/classes java/org/tigris/scarab/om

Author: ronvoe122
Date: 2007-06-10 06:48:09-0700
New Revision: 10448

Modified:
   trunk/src/conf/classes/cache.ccf
   trunk/src/java/org/tigris/scarab/om/ActivitySet.java
   trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java

Log:
[SCB1691] cache the activities in an ActivitySet. Reduces the number of db queries when viewing the history of an issue.

Modified: trunk/src/conf/classes/cache.ccf
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/conf/classes/cache.ccf?view=diff&rev=10448&p1=trunk/src/conf/classes/cache.ccf&p2=trunk/src/conf/classes/cache.ccf&r1=10447&r2=10448
==============================================================================
--- trunk/src/conf/classes/cache.ccf	(original)
+++ trunk/src/conf/classes/cache.ccf	2007-06-10 06:48:09-0700
@@ -105,4 +105,8 @@
 jcs.region.org_tigris_scarab_om_DependType.cacheattributes.MemoryCacheName=org.apache.jcs.engine.memory.lru.LRUMemoryCache

 jcs.region.org_tigris_scarab_om_DependType.elementattributes=org.apache.jcs.engine.ElementAttributes

 

-@JCS_CACHE_PROPERTIES@

+jcs.region.org_tigris_scarab_om_ActivitySet=

+jcs.region.org_tigris_scarab_om_ActivitySet.cacheattributes=org.apache.jcs.engine.CompositeCacheAttributes

+jcs.region.org_tigris_scarab_om_ActivitySet.cacheattributes.MaxObjects=500

+jcs.region.org_tigris_scarab_om_ActivitySet.cacheattributes.MemoryCacheName=org.apache.jcs.engine.memory.lru.LRUMemoryCache

+jcs.region.org_tigris_scarab_om_ActivitySet.elementattributes=org.apache.jcs.engine.ElementAttributes


Modified: trunk/src/java/org/tigris/scarab/om/ActivitySet.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/ActivitySet.java?view=diff&rev=10448&p1=trunk/src/java/org/tigris/scarab/om/ActivitySet.java&p2=trunk/src/java/org/tigris/scarab/om/ActivitySet.java&r1=10447&r2=10448
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/ActivitySet.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/ActivitySet.java	2007-06-10 06:48:09-0700
@@ -133,10 +133,20 @@
      */
     public List getActivityListForIssue(Issue issue) throws TorqueException
     {
+         List activityList = (List)ActivitySetManager.getMethodResult()
+                .get(this, "getActivityListForIssue", issue );
+
+        if(activityList==null)
+        {
             Criteria crit = new Criteria()
                 .add(ActivityPeer.TRANSACTION_ID, getActivitySetId());
             crit.add(ActivityPeer.ISSUE_ID, issue.getIssueId());
-            return ActivityPeer.doSelect(crit);
+            activityList = ActivityPeer.doSelect(crit);
+
+            ActivitySetManager.getMethodResult()
+                .put(activityList, this, "getActivityListForIssue", issue );
+        }
+        return activityList;  
     }
 
     public ScarabUser getCreator()

Modified: trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java?view=diff&rev=10448&p1=trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java&p2=trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java&r1=10447&r2=10448
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/ActivitySetManager.java	2007-06-10 06:48:09-0700
@@ -73,6 +73,7 @@
         throws TorqueException
     {
         super();
+        setRegion(getClassName().replace('.', '_'));
     }
 
     /**