[jira] [Commented] (KARAF-4748) Make Felix Resolver Threads configurable

["Fabian Lange (JIRA)" <ji...@apache.org>]

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

Fabian Lange commented on KARAF-4748:
-------------------------------------

[~gnt] what do you think? The Felix change has been merged, but the problem remains for me. I really would like to control this somehow. Could we allow

        Resolver resolver = new ResolverImpl(new Slf4jResolverLog(LoggerFactory.getLogger(ResolverImpl.class)));

to take the additional new parallelism argument?

> Make Felix Resolver Threads configurable
> ----------------------------------------
>
>                 Key: KARAF-4748
>                 URL: https://issues.apache.org/jira/browse/KARAF-4748
>             Project: Karaf
>          Issue Type: Bug
>            Reporter: Fabian Lange
>         Attachments: Screen Shot 2016-10-03 at 20.13.47.png, Screen Shot 2016-10-04 at 09.25.48.png, Screen Shot 2016-10-04 at 09.26.04.png, Screen Shot 2016-10-04 at 09.26.17.png, Screen Shot 2016-10-04 at 09.27.45.png, Screen Shot 2016-10-04 at 09.28.20.png, Screen Shot 2016-10-04 at 09.28.50.png, Screen Shot 2016-10-04 at 09.30.14.png, Screen Shot 2016-10-04 at 09.31.16.png, Screen Shot 2016-10-04 at 11.54.19.png
>
>
> It was requested in https://issues.apache.org/jira/browse/FELIX-5247 to make the resolver threads configurable to avoid creating many threads by karaf.
> It turns out Felix already has that feature, but Karaf is not using it:
> https://github.com/apache/karaf/blob/karaf-4.0.x/features/core/src/main/java/org/apache/karaf/features/internal/osgi/Activator.java#L119
> Resolver resolver = new ResolverImpl(new Logger(Logger.LOG_INFO));
> I propose to configure and supply an Executor with the desired threads. As additional up, this prevents situations where there are unnecessary threads created.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)