[jira] Commented: (FELIX-1416) Wrong factory configuration behaviour

["Felix Meschberger (JIRA)" <ji...@apache.org>]

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

Felix Meschberger commented on FELIX-1416:
------------------------------------------

Added unit and integration (Pax Exam based) tests for the fixed configuration behaviour in Rev. 798522

In Rev. 798529 adapted the ImmediateComponentManager to the new holder mechanism.

Rev. 798531 finally brings the big block of ComponentHolder and implementation.

The ComponentRegistry is refactored to be a factory for ComponentHolders and has no Configuration support. The ConfigurationComponentRegistry extends the ComponentRegistry adding support for Configuration from ConfigurationAdmin.

For ComponentFactory components the ComponentFactoryImpl also acts as the component holder. For Components without configuration (configuration-policy=ignore) UnconfiguredComponentHolder instances are used. This class is also used if no ConfigurationAdmin service is available.

For components taking configuration the ConfiguredComponentHolder class is used. This class can cope with singleton and factory configurations.

The Configuration policy "ignore" is handled by the ComponentRegistry in that an UnconfiguredComponentHolder is created to hold the component without ever forwarding configuration.

For Configuration policy "require" and "optional" a ConfiguredComponentHolder is used. The "require" policy is handled by the AbstractComponentManager to prevent activation if there is no configuration.

> Wrong factory configuration behaviour
> -------------------------------------
>
>                 Key: FELIX-1416
>                 URL: https://issues.apache.org/jira/browse/FELIX-1416
>             Project: Felix
>          Issue Type: Bug
>          Components: Declarative Services (SCR), Specification compliance
>    Affects Versions: scr-1.0.8
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: scr-1.2.0
>
>
> Currently factory configurations are applied to component factories, such that each factory configuration instance creates a componnent
> instances of a component factory. Reversly deleting a factory configuration also deletes component instances. This is not how it is specified.
> Correct is, that 
>   (1) Component Factories can only be configured with singleton configurations applying
>           the configuration to all instances created with newInstance
>   (2) Factory configurations are applied to non-component-factory components and
>           cause multiple component instances to be created.
> To accomodate for this, the handling of components has to be redesigned: A component descriptor now causes the creation of a ComponentHolder. Depending on configuration availability a ComponentHolder will hold a single component (no configuration or singleton configuration) or multiple components (factory configuration).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.