[infinispan-issues] [JBoss JIRA] (ISPN-3849) Re-work ServiceLoader concepts

[ https://issues.jboss.org/browse/ISPN-3849?page=com.atlassian.jira.plugin.... ] Tuomas Kiviaho commented on ISPN-3849: -------------------------------------- I pinpoint here that bundle providing ServiceLoader must be in state ACTIVE. Otherwise you'll end up with NPE because BundleContext will be missing. {code} private static <T> void addOsgiServices(Class<T> contract, Set<T> services) { if (!Util.isOSGiContext()) { return; } ClassLoader loader = ServiceFinder.class.getClassLoader(); if ((loader != null) && (loader instanceof org.osgi.framework.BundleReference)) { final BundleContext bundleContext = ((BundleReference) loader).getBundle().getBundleContext(); final ServiceTracker<T, T> serviceTracker = new ServiceTracker<T, T>(bundleContext, contract.getName(), null); serviceTracker.open(); try { final Object[] osgiServices = serviceTracker.getServices(); if (osgiServices != null) { for (Object osgiService : osgiServices) { services.add((T) osgiService); } } } catch (Exception e) { // ignore } } } {code} A far better approach might be to ditch ServiceTracker altogether in favor of just handpicking the service reference because that class requires OSGi Compendium to be present and here it's not necessarily needed for what is being done. Side note: ServiceReference objects that are also inside ServiceTracker should always be unget after service is no longer needed. Hence the open doesn't have close companion which contradicts the WhiteBoard pattern. -- This message was sent by Atlassian JIRA (v7.2.3#72005)