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

Richard S. Hall commented on FELIX-884:
---------------------------------------

Ok, I forgot I implemented that way, I just assumed we would cache it, but I 
see in my comments that caching is a "to do". Given that it is not long lived, 
I agree it is probably ok as is or possibly better with a service listener to 
invalidate the service state.

The local repo is disposed every time Resolver.resolve() is called, so it does 
probably make more sense for us to dispose it at the end of the resolve() 
method.

Is there any value in adding the service capabilities directly to the 
appropriate bundle using getRegisteredServices() as we process the bundles?

> OBR should expose registered services as capabilities of local repository
> -------------------------------------------------------------------------
>
>                 Key: FELIX-884
>                 URL: https://issues.apache.org/jira/browse/FELIX-884
>             Project: Felix
>          Issue Type: Improvement
>          Components: Bundle Repository (OBR)
>    Affects Versions: bundlerepository-1.2.0
>            Reporter: Felix Meschberger
>         Attachments: FELIX-884.patch
>
>
> The OBR implementation exposes the current state of the framework as a 
> Repository, the LocalRepository. This provides the installed bundles as parts 
> provided by a repository with their capabilities. These capabilities do not 
> currently only reflect the currently installed set of bundles and their 
> static setup such as package import/export and service import/export. The 
> actually registered services, though are not exposed as capabilities 
> whatsoever.
> So if for example, the OPS4J Pax Web Service bundle registers an HttpService 
> instance. But this fact is not reflected by a ExportService header. Trying to 
> install a bundle from the OBR which requires an HttpService fails, since the 
> actually registered HttpService is not provided as a capability.
> I suggest, the LocalRepository exposes all registered services (just their 
> names would probably do it) as capabilities of the system bundle. This is not 
> entirely correct but should probably be enough to satisfy the requirements of 
> bundles.
> WDYT ?
> Ref: Discussion on user list at http://markmail.org/message/ivkdvhpk4gclkylv

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

Reply via email to