[ 
https://issues.apache.org/jira/browse/FELIX-3577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13405687#comment-13405687
 ] 

Felix Meschberger commented on FELIX-3577:
------------------------------------------

Rev. 1356293 Fix a conversion glitch (calling service reference removal instead 
of configuration removal). Also renamed the provide and remove methods to 
provideConfiguration and removeConfiguration to make the distinction of removal 
clear.
                
> Refactor helpers and service trackers
> -------------------------------------
>
>                 Key: FELIX-3577
>                 URL: https://issues.apache.org/jira/browse/FELIX-3577
>             Project: Felix
>          Issue Type: Improvement
>          Components: Configuration Admin
>    Affects Versions: configadmin-1.4.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: configadmin-1.6.0
>
>
> The ConfigurationManager class currently hosts quite a number of inner 
> classes to help with its tasks. A few of these classes have overlapping 
> functionality which makes it hard to easily implement targeted PIDs these 
> functionalities should be better aligned:
>  * ManagedService[Factory]Tracker don't really get the services but maintain 
> a map of PIDs and assigned
>       configurations (most importantly the targeted PID and configuration 
> modification counter)
>  * ManagedService[Factory] callbacks are also executed by the tracker classes
>  * The ManagedService[Factory]Helper classes are renamed to 
> [Factory]ConfigurationHelper since
>        they are helpers to convey configuration to ManagedService[Factory] 
> services through the respective
>        trackers.
>  * The ServiceHolder class is replaced by the ConfigurationMap class managed 
> by the trackers.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to