Uploaded image for project: 'Cache Modules'
  1. Cache Modules
  2. MGNLCACHE-78

Move store caching configuration into cachePolicy

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Neutral
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.4
    • Component/s: None
    • Labels:
    • Magnolia Release:
      5.4

      Description

      MGNLCACHE-57 introduced isCacheableVoters configuration under Default cache policy to have all cache voters at one place:

      1. Move this configuration to CachePolicy interface next to the voters for bypass. I'd also suggest to make it more powerful than just cacheable/noncacheable. I could return TTL and the logic from info.magnolia.module.cache.executor.Store#getTimeToLive should go there.
      2. Rename CachePolicy/voters to something more explanatory (bypass/shouldBypass).
      3. -Move also other configuration there that's spread across other places (e.g Store also has a compressible configuration item)- this is no longer the case

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              rkovarik Roman Kovařík
              Reporter:
              rkovarik Roman Kovařík
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Date of First Response: