You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by dj...@apache.org on 2016/01/27 02:48:08 UTC

svn commit: r1726931 - in /felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config: ConfigurableComponentHolder.java RegionConfigurationSupport.java

Author: djencks
Date: Wed Jan 27 01:48:08 2016
New Revision: 1726931

URL: http://svn.apache.org/viewvc?rev=1726931&view=rev
Log:
FELIX-5174 log configuration change count when a configuration is processed

Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/ConfigurableComponentHolder.java
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/RegionConfigurationSupport.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/ConfigurableComponentHolder.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/ConfigurableComponentHolder.java?rev=1726931&r1=1726930&r2=1726931&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/ConfigurableComponentHolder.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/ConfigurableComponentHolder.java Wed Jan 27 01:48:08 2016
@@ -344,8 +344,8 @@ public class ConfigurableComponentHolder
      */
     public boolean configurationUpdated( TargetedPID pid, TargetedPID factoryPid, final Dictionary<String, Object> props, long changeCount )
     {
-        log( LogService.LOG_DEBUG, "ConfigurableComponentHolder configuration updated for pid {0} with properties {1}",
-            new Object[] {pid, props}, null);
+        log( LogService.LOG_DEBUG, "ConfigurableComponentHolder configuration updated for pid {0} with properties {1} and change count {2}",
+            new Object[] {pid, props, changeCount}, null);
 
         // component to update or create
         final Map<AbstractComponentManager<S>, Map<String, Object>> scms = new HashMap< AbstractComponentManager<S>, Map<String, Object>>();

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/RegionConfigurationSupport.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/RegionConfigurationSupport.java?rev=1726931&r1=1726930&r2=1726931&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/RegionConfigurationSupport.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/config/RegionConfigurationSupport.java Wed Jan 27 01:48:08 2016
@@ -170,8 +170,8 @@ public class RegionConfigurationSupport
                                     for ( Configuration config: factory )
                                     {
                                         Activator.log( LogService.LOG_DEBUG, null,
-                                                "Configuring holder {0} with factory configuration {1}", new Object[] {
-                                                        holder, config }, null );
+                                                "Configuring holder {0} with factory configuration {1}, change count {2}", new Object[] {
+                                                        holder, config, config.getChangeCount() }, null );
                                         config = getConfiguration( ca, config.getPid() );
                                         if ( checkBundleLocation( config, bundleContext.getBundle() ) )
                                         {
@@ -196,8 +196,8 @@ public class RegionConfigurationSupport
                                     {
                                         singleton = getConfiguration( ca, singleton.getPid() );
                                         Activator.log( LogService.LOG_DEBUG, null,
-                                                "Configuring holder {0} with configuration {1}", new Object[] { holder,
-                                                        singleton }, null );
+                                                "Configuring holder {0} with configuration {1}, change count {2}", new Object[] { holder,
+                                                        singleton, singleton.getChangeCount() }, null );
                                         if ( singleton != null
                                                 && checkBundleLocation( singleton, bundleContext.getBundle() ) )
                                         {
@@ -360,8 +360,8 @@ public class RegionConfigurationSupport
                         final ConfigurationInfo configInfo = getConfigurationInfo( pid, targetedPid, componentHolder, bundleContext );
                         if ( configInfo != null )
                         {
-                            Activator.log(LogService.LOG_DEBUG, null, "LocationChanged event, same targetedPID {0}, location now {1}",
-                                    new Object[] {targetedPid, configInfo.getBundleLocation()},
+                            Activator.log(LogService.LOG_DEBUG, null, "LocationChanged event, same targetedPID {0}, location now {1}, change count {2}",
+                                    new Object[] {targetedPid, configInfo.getBundleLocation(), configInfo.getChangeCount()},
                                     null);
                             if (configInfo.getProps() == null)
                             {
@@ -389,8 +389,8 @@ public class RegionConfigurationSupport
                         final ConfigurationInfo configInfo = getConfigurationInfo( pid, targetedPid, componentHolder, bundleContext );
                         if ( configInfo != null )
                         {
-                            Activator.log(LogService.LOG_DEBUG, null, "LocationChanged event, better targetedPID {0} compared to {1}, location now {2}",
-                                    new Object[] {targetedPid, oldTargetedPID, configInfo.getBundleLocation()},
+                            Activator.log(LogService.LOG_DEBUG, null, "LocationChanged event, better targetedPID {0} compared to {1}, location now {2}, change count {3}",
+                                    new Object[] {targetedPid, oldTargetedPID, configInfo.getBundleLocation(), configInfo.getChangeCount()},
                                     null);
                             if (configInfo.getProps() == null)
                             {