Uploaded image for project: 'Felix'
  1. Felix
  2. FELIX-1223

Replace ManagedService[Factory] by ConfigurationListener based component configuration

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • scr-1.0.8
    • scr-1.2.0
    • None

    Description

      Currently configuration is provided to components by registering a ManagedService services for each non-factory component and ManagedServiceFactory services for each factory component. This has several drawbacks:

      • It may lead to component cycling on startup: start with no configuration, get configuration after activation and this reactivate
      • As a consequence of unneeded component reactivation, the system is loaded more and changes for deadlocks raise
      • As a consequence of unneeded component reactivation, other components might need to be reactivated, too, further raising load
      • It places a load on the service registry due to the number for registered ManagedService[Factory] services
      • It makes it probably harder to implement FELIX-924

      As discussed on the dev list [1] it would be better to refactor configuration support as follows:

      • When a component is created, the existing configuration is already retrieved from ConfigurationAdmin.
      • SCR registers a ConfigurationListener which updates the components on configuration updates.

      Primarily the first step allows for simple FELIX-924 implementation in that on component activation, we may ensure configuration presence if required. Also it removes the requirement to reactivate components on startup since configuration events are only sent upon effective configuration change.

      Attachments

        Activity

          People

            fmeschbe Felix Meschberger
            fmeschbe Felix Meschberger
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: