svn commit: r1593764 - in /axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment: OSGiConfigurationContextFactory.java ServiceRegistry.java

[veithen]

Author: veithen
Date: Sun May 11 09:37:43 2014
New Revision: 1593764

URL: http://svn.apache.org/r1593764
Log:
Improved logging.

Modified:
    
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/OSGiConfigurationContextFactory.java
    
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/ServiceRegistry.java

Modified: 
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/OSGiConfigurationContextFactory.java
URL: 
http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/OSGiConfigurationContextFactory.java?rev=1593764&r1=1593763&r2=1593764&view=diff
==============================================================================
--- 
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/OSGiConfigurationContextFactory.java
 (original)
+++ 
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/OSGiConfigurationContextFactory.java
 Sun May 11 09:37:43 2014
@@ -77,6 +77,7 @@ public class OSGiConfigurationContextFac
         }
         bundleTracker.close();
         if (configCtx != null) {
+            log.debug("Terminating configuration context");
             try {
                 configCtx.terminate();
                 configCtx = null;

Modified: 
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/ServiceRegistry.java
URL: 
http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/ServiceRegistry.java?rev=1593764&r1=1593763&r2=1593764&view=diff
==============================================================================
--- 
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/ServiceRegistry.java
 (original)
+++ 
axis/axis2/java/core/trunk/modules/osgi/src/org/apache/axis2/osgi/deployment/ServiceRegistry.java
 Sun May 11 09:37:43 2014
@@ -174,17 +174,19 @@ public class ServiceRegistry extends Abs
                         unreslovedBundles.add(bundle);
                     }
                     try {
-                        for (Iterator iterator = 
axisServiceGroup.getServices();
-                             iterator.hasNext();) {
-                            AxisService service = (AxisService) 
iterator.next();
-                            log.info("[Axis2/OSGi] Service - " + 
service.getName());
+                        if (log.isInfoEnabled()) {
+                            List<String> serviceNames = new 
ArrayList<String>();
+                            for (Iterator<AxisService> iterator = 
axisServiceGroup.getServices();
+                                    iterator.hasNext(); ) {
+                                AxisService service = (AxisService) 
iterator.next();
+                                serviceNames.add(service.getName());
+                            }
+                            log.info("Stopping service group "
+                                    + axisServiceGroup.getServiceGroupName() + 
" with services "
+                                    + serviceNames + " in bundle " + 
bundle.getSymbolicName());
                         }
                         configCtx.getAxisConfiguration()
                                 
.removeServiceGroup(axisServiceGroup.getServiceGroupName());
-                        log.info("[Axis2/OSGi] Stopping " +
-                                 axisServiceGroup.getServiceGroupName() +
-                                 " service group in Bundle - " +
-                                 bundle.getSymbolicName());
                     } catch (AxisFault e) {
                         String msg = "Error while removing the service group";
                         log.error(msg, e);