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

Move store caching configuration into cachePolicy

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Neutral
    • 5.4
    • None
    • None

    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

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD