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

Refactor CacheIgnoreManager - AbstractProtocolServer collaboration

    XMLWordPrintable

Details

    • Enhancement
    • Resolution: Done
    • Major
    • 10.1.4.Final, 11.0.0.Dev03
    • 10.1.0.Final
    • Server
    • None

    Description

      Given the interactions between CacheIgnoreManager and AbstractProtocolServer it seems that CacheIgnoreManager should be a global component in the registry of the EmbeddedCacheManager sitting behind the AbstractProtocolServer. It should be looked up from there, or better injected, rather than being received in ProtocolServer.start method. The start method also has an overloaded version that does not take a CacheIgnoreManager to support tests the do not need it. This lack of orthogonality is annoyoing.

      Another aspect to refactor: the startInternal method receives parametters that it assigns to fields. It seems that this should have been done already in the start method. That would get rid of some clunky hacks and some comments/excuses for the said hacks.

      Attachments

        Activity

          People

            anistor Adrian Nistor (Inactive)
            anistor Adrian Nistor (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: