Log message improvements in ResourceBatchingManager 48/43048/2
authorHANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Wed, 3 Aug 2016 14:18:39 +0000 (19:48 +0530)
committerhanamantagoud <hanamantagoud.v.kandagal@ericsson.com>
Thu, 4 Aug 2016 03:09:40 +0000 (03:09 +0000)
Change-Id: I43fadc6d32bbeccedbaf51c59599f78b847b32a7
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/utils/batching/ResourceBatchingManager.java

index cd3e543e10cabd6cca49531c45ed6bb6f1d6b818..4aa5f00f1abd5dc6612f138060966e78057a22df 100644 (file)
@@ -93,9 +93,9 @@ public class ResourceBatchingManager implements AutoCloseable {
                 int batchSize = resHandler.getBatchSize();
 
                 int batches = resList.size()/ batchSize;
-                LOG.trace("Picked up size {} batches {}", resList.size(), batches);
                 if ( resList.size() > batchSize)
                 {
+                    LOG.info("Batched up resources of size {} into batches {} for resourcetype {}", resList.size(), batches, resourceType);
                     for (int i = 0, j = 0; i < batches; j = j + batchSize,i++)
                     {
                         new MdsalDsTask<>(resourceType, resList.subList(j, j + batchSize)).process();
@@ -110,7 +110,7 @@ public class ResourceBatchingManager implements AutoCloseable {
                 }
 
                 long timetaken = System.currentTimeMillis() - start;
-                LOG.info( "Total time taken for resourceList of size: " + resList.size() + " ### time =  " + timetaken);
+                LOG.info( "Total taken ##time = {}ms for resourceList of size {} for resourceType {}", timetaken, resList.size(), resourceType);
 
             } catch (InterruptedException e)
             {
@@ -178,7 +178,7 @@ public class ResourceBatchingManager implements AutoCloseable {
                     futures.get();
                     long time = System.currentTimeMillis() - start;
 
-                    LOG.info( " ##### Time taken for " + actResourceList.size() + " = " + time);
+                    LOG.trace( " ##### Time taken for " + actResourceList.size() + " = " + time + "ms");
 
                 } catch (InterruptedException | ExecutionException e)
                 {