Uploaded image for project: 'Infinispan'
  1. Infinispan
  2. ISPN-13241

Remove InternalExpirationManager.entryExpiredInMemoryFromIteration

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Done
    • 13.0.0.Dev03, 12.1.7.Final
    • 13.0.0.CR1
    • Core, Expiration
    • None

    Description

      InternalExpirationManager.entryExpiredInMemoryFromIteration() is currently implemented to either:

      • Synchronously remove the expired entry in local caches
      • Skip the expiration check and return expired transient (max-idle) entries in clustered caches

      The local cache (ExpirationManagerImpl) implementation is perhaps useful to speed up repeated iterations over the cache, but the downside is that it will block to wait for store removals.

      The clustered cache (ClusteredExpirationManagerImpl implementation was needed back when confirming the expiration status of an entry required a RPC, but that is no longer the case.

      We can just remove entryExpiredInMemoryFromIteration() and any calls to it from the iterators, and it should only break tests that make assumptions about the expiration implementation.

      Attachments

        Activity

          People

            dberinde@redhat.com Dan Berindei
            dberinde@redhat.com Dan Berindei
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: