Pluto
  1. Pluto
  2. PLUTO-539

New FilterManagerService implementation

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Labels:
      None

      Description

      Below copied (in part) from email discussion on the Pluto dev list, see also: http://www.nabble.com/More-required-Pluto-2.0-SPI-and-implementation-refactoring-issues-td21973310.html

          • FilterManager implementation and usage
            The current implementation and usage of the FilterManager is very inefficient: a new FilterManagerImpl and FilterChainImpl is created on the
            fly for each container invocation, while the behavior and context of a PortletFilter really is deployment descriptor based.
            This implementation needs to be refactoring to a much more performant solution.

      =====
      I've added a new (required) FilterManagerService which will allow to cache a FilterChain per portlet application per FilterManager

        Activity

        Hide
        Ate Douma added a comment -

        I've implemented this on portal-driver by simply returning a new FilterManagerImpl from the new service.
        No optimization done yet, but at least its now integrated with the new container api again.
        Further optimization is (for the portal-driver) not a critical requirement right now.

        Show
        Ate Douma added a comment - I've implemented this on portal-driver by simply returning a new FilterManagerImpl from the new service. No optimization done yet, but at least its now integrated with the new container api again. Further optimization is (for the portal-driver) not a critical requirement right now.

          People

          • Assignee:
            Ate Douma
            Reporter:
            Ate Douma
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development