[
https://issues.apache.org/jira/browse/JCLOUDS-85?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13698041#comment-13698041
]
Andrew Phillips commented on JCLOUDS-85:
----------------------------------------
> only if no injection takes place
When does the injection take place? I.e. is there a way to determine whether we
need to instantiate the class *without* having to check in every "execute"
call, as the current PR does?
> Avoid creating uneeded instance of PropertyShellTableFactory
> ------------------------------------------------------------
>
> Key: JCLOUDS-85
> URL: https://issues.apache.org/jira/browse/JCLOUDS-85
> Project: jclouds
> Issue Type: Improvement
> Components: jclouds-karaf
> Affects Versions: 1.7.0, 1.6.1, 1.5.11
> Reporter: Ioannis Canellos
>
> The story here is that we have 2 versions of the ShellTableFactory:
> i) OSGi aware
> ii) Property based
> Our commands always instantiate (ii) and may replace it with (i) when running
> inside OSGi.
> We should make sure that we instantiate (ii) only if no injection takes place
> (e.g. outside OSGi)
> This could be as simple as checking if the factory is null inside the
> doExecute method and then instantiating (ii).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira