You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pd...@apache.org on 2015/05/17 00:29:19 UTC

svn commit: r1679813 - /felix/sandbox/pderop/bndtools-FELIX-4866/org.apache.felix.framework.concurrencytest/src/org/apache/felix/framework/concurrencytest/Loader.java

Author: pderop
Date: Sat May 16 22:29:19 2015
New Revision: 1679813

URL: http://svn.apache.org/r1679813
Log:
Don't close service trackers in case of errors, in order to be able to inspect service registry from gogo shell.

Modified:
    felix/sandbox/pderop/bndtools-FELIX-4866/org.apache.felix.framework.concurrencytest/src/org/apache/felix/framework/concurrencytest/Loader.java

Modified: felix/sandbox/pderop/bndtools-FELIX-4866/org.apache.felix.framework.concurrencytest/src/org/apache/felix/framework/concurrencytest/Loader.java
URL: http://svn.apache.org/viewvc/felix/sandbox/pderop/bndtools-FELIX-4866/org.apache.felix.framework.concurrencytest/src/org/apache/felix/framework/concurrencytest/Loader.java?rev=1679813&r1=1679812&r2=1679813&view=diff
==============================================================================
--- felix/sandbox/pderop/bndtools-FELIX-4866/org.apache.felix.framework.concurrencytest/src/org/apache/felix/framework/concurrencytest/Loader.java (original)
+++ felix/sandbox/pderop/bndtools-FELIX-4866/org.apache.felix.framework.concurrencytest/src/org/apache/felix/framework/concurrencytest/Loader.java Sat May 16 22:29:19 2015
@@ -66,13 +66,13 @@ public class Loader implements Runnable
 		// Count the number of satisfied components.
 		long satisfied = components.stream().filter(Component::isSatisfied).count();			
 		
-		// Stop all components
-        components.stream().forEach(Component::stop);
-        
         // Report an error if we don't have expected satisfied components.
 		if (satisfied != COMPONENTS) {
 			throw new Exception("found unsatisfied component: " + (COMPONENTS - satisfied));			
 		}
+
+		// Stop all components
+        components.stream().forEach(Component::stop);        
 	}
 
 	private Component createComponents(int i) {