Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-38459

Cache configuration format memory.<cachename> = doesn't work for all caches

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.9.x
    • Fix Version/s: 10.0
    • Component/s: Kernel
    • Labels:
      None
    • Previous Issue Keys:
      KNL-1118
    • Test Plan:
      Hide

      To test, you'd set a property like
      memory.org.sakaiproject.db.BaseDbFlatStorage.SAKAI_SITE_PAGE_PROPERTY=maxElementsInMemory=1234,timeToIdleSeconds=1234

      Then go to Administration Workspace->Memory->Status and see if the values were changed on the bottom for that cache to 1234. (Not an optional value but easy to search that it changed)

      Ideally we'd test every cache available on that page to make sure they can be overridden.

      Show
      To test, you'd set a property like memory.org.sakaiproject.db.BaseDbFlatStorage.SAKAI_SITE_PAGE_PROPERTY=maxElementsInMemory=1234,timeToIdleSeconds=1234 Then go to Administration Workspace->Memory->Status and see if the values were changed on the bottom for that cache to 1234. (Not an optional value but easy to search that it changed) Ideally we'd test every cache available on that page to make sure they can be overridden.

      Description

      If you set the property for some caches like documented, it doesn't work, you have to use bean style properties to set these. This was moved and reorganized in KNL-532. This is because these are going through EhCacheFactoryBean rather than BasicMemoryService. I'm committing some fixes to a branch where I got this working but would really like review. It uses a class CacheInitializer which EhCacheFactoryBean can't get to directly being in impl (Not without the api proxy right?) so I had to move that.

      memory.org.sakaiproject.db.BaseDbFlatStorage.SAKAI_SITE_PAGE_PROPERTY=maxElementsInMemory=50000

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  jonespm Matthew Jones
                  Reporter:
                  jonespm Matthew Jones
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration