Grouper Users - Open Discussion List

[Chris Hyzer <>]

I added an API unit test which fails the old way and passes the new way, added some debug info, and updated the patch on the jira, use that one.

 

Thanks,

Chris

 

From: [mailto:] On Behalf Of Chris Hyzer
Sent: Friday, January 13, 2012 7:00 PM
To:
Subject: [grouper-users] RE: issue with lite ui 2.0.3

 

Ok, this is not as big of a deal as I originally thought.  It probably will not affect you, one place it will affect you is if you customize the lite UI to only allow certain subject sources to be returned for subject searches (something we do at Penn)…

 

https://bugs.internet2.edu/jira/browse/GRP-725

 

This is fixed, tweak in CachingResolver. Note that this is only a problem on searches when multiple sources are specified, which is rare, at Penn this happened in our SubjectPicker screen where we specify that the sources are PennPerson or groups (not external people or kerberos principals). Patch:

 

### Eclipse Workspace Patch 1.0

#P grouper

Index: src/grouper/edu/internet2/middleware/grouper/subj/CachingResolver.java

===================================================================

--- src/grouper/edu/internet2/middleware/grouper/subj/CachingResolver.java             (revision 7747)

+++ src/grouper/edu/internet2/middleware/grouper/subj/CachingResolver.java           (working copy)

@@ -608,7 +608,7 @@

     fullKey[1] = grouperSessionSubject.getId();

     fullKey[2] = stemName;

     fullKey[3] = query;

-    System.arraycopy(sourcesArray, 0, fullKey, 2, sourcesArray.length);

+    System.arraycopy(sourcesArray, 0, fullKey, 4, sourcesArray.length);

     MultiKey multiKey = new MultiKey(fullKey);

     return multiKey;

   }

 

 

 

From: Chris Hyzer
Sent: Thursday, January 12, 2012 4:23 PM
To:
Subject: issue with lite ui 2.0.3

 

https://bugs.internet2.edu/jira/browse/GRP-725

 

There was a subject search caching issue with the lite ui combobox which made it difficult for users to add people to groups. I changed the grouper.ehcache.xml config file so that the caching is not used (1 second), which fixes the problem. I will look through the code to find out why this occurs and what the real solution is. (Note, I dont know which subject searches/caches are affected yet, so to be safe I set all to 1, but it is at least FindPage) 

  <!-- @see CachingResolver#find(...) --> 
  <cache name="edu.internet2.middleware.grouper.subj.CachingResolver.Find" 
          maxElementsInMemory="5000" 
          eternal="false" 
          timeToIdleSeconds="1" 
          timeToLiveSeconds="1" 
          overflowToDisk="false" 
  /> 

  <!-- @see CachingResolver#findAll(...) --> 
  <cache name="edu.internet2.middleware.grouper.subj.CachingResolver.FindAll" 
          maxElementsInMemory="5000" 
          eternal="false" 
          timeToIdleSeconds="1" 
          timeToLiveSeconds="1" 
          overflowToDisk="false" 
  /> 

  <!-- @see CachingResolver#findPage(...) --> 
  <cache name="edu.internet2.middleware.grouper.subj.CachingResolver.FindPage" 
          maxElementsInMemory="5000" 
          eternal="false" 
          timeToIdleSeconds="1" 
          timeToLiveSeconds="1" 
          overflowToDisk="false" 
  /> 

  <!-- @see CachingResolver#findByIdentifier(...) --> 
  <cache name="edu.internet2.middleware.grouper.subj.CachingResolver.FindByIdentifier" 
          maxElementsInMemory="5000" 
          eternal="false" 
          timeToIdleSeconds="1" 
          timeToLiveSeconds="1" 
          overflowToDisk="false" 
  /> 

  <!-- @see CachingResolver#findByIdOrIdentifier(...) --> 
  <cache name="edu.internet2.middleware.grouper.subj.CachingResolver.FindByIdOrIdentifier" 
          maxElementsInMemory="5000" 
          eternal="false" 
          timeToIdleSeconds="1" 
          timeToLiveSeconds="1" 
          overflowToDisk="false" 
  /> 


Thanks, 
Chris