Remove logging at INFO level in repo-common

These logging instructions were only temporary (during development).
Evolveum · Jul 7, 2021 · e45d721 · e45d721
1 parent 42da136
commit e45d721
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 12 deletions.
diff --git a/...n/src/main/java/com/evolveum/midpoint/repo/common/activity/ActivityTreeStateOverview.java b/...n/src/main/java/com/evolveum/midpoint/repo/common/activity/ActivityTreeStateOverview.java
@@ -152,7 +152,7 @@ public void purge(OperationResult result) throws ActivityExecutionException {
                 ),
                 result);
 
-        LOGGER.info("State tree after purging: {}", lazy(this::getActivityStateTree)); // TODO trace
+        LOGGER.trace("State tree after purging: {}", lazy(this::getActivityStateTree));
     }
 
     private ActivityStateOverviewType purgeStateRecursively(ActivityStateOverviewType state) {

diff --git a/.../java/com/evolveum/midpoint/repo/common/activity/execution/AbstractActivityExecution.java b/.../java/com/evolveum/midpoint/repo/common/activity/execution/AbstractActivityExecution.java
@@ -232,21 +232,21 @@ private void completeExecutionResult(ActivityExecutionResult executionResult) {
         }
     }
 
-    private void logStart() { // todo debug
-        LOGGER.info("{}: Starting execution of activity with identifier '{}' and path '{}' (local: '{}') with work state "
+    private void logStart() {
+        LOGGER.debug("{}: Starting execution of activity with identifier '{}' and path '{}' (local: '{}') with work state "
                         + "prism item path: {}",
                 getClass().getSimpleName(), activity.getIdentifier(), activity.getPath(), activity.getLocalPath(),
                 activityState.getItemPath());
     }
 
-    private void logEnd(ActivityExecutionResult executionResult) { // todo debug
-        LOGGER.info("{}: Finished execution of activity with identifier '{}' and path '{}' (local: {}) with result: {}",
+    private void logEnd(ActivityExecutionResult executionResult) {
+        LOGGER.debug("{}: Finished execution of activity with identifier '{}' and path '{}' (local: {}) with result: {}",
                 getClass().getSimpleName(), activity.getIdentifier(), activity.getPath(), activity.getLocalPath(),
                 executionResult);
     }
 
-    private void logComplete() { // todo debug
-        LOGGER.info("{}: Skipped execution of activity with identifier '{}' and path '{}' (local: {}) as it was already executed",
+    private void logComplete() {
+        LOGGER.debug("{}: Skipped execution of activity with identifier '{}' and path '{}' (local: {}) as it was already executed",
                 getClass().getSimpleName(), activity.getIdentifier(), activity.getPath(), activity.getLocalPath());
     }
 

diff --git a/...ava/com/evolveum/midpoint/repo/common/activity/execution/DelegatingActivityExecution.java b/...ava/com/evolveum/midpoint/repo/common/activity/execution/DelegatingActivityExecution.java
@@ -160,7 +160,7 @@ private Task createSuspendedChildTask(OperationResult result) throws ActivityExe
                     .localRoot(localRoot.toBean())
                     .localRootActivityExecutionRole(ActivityExecutionRoleType.DELEGATE);
 
-            LOGGER.info("Creating activity subtask {} with local root {}", childToCreate.getName(), localRoot);
+            LOGGER.debug("Creating activity subtask {} with local root {}", childToCreate.getName(), localRoot);
             String childOid = getBeans().taskManager.addTask(childToCreate.asPrismObject(), result);
             LOGGER.debug("Created activity subtask {}: {}", childToCreate.getName(), childOid);
 

diff --git a/.../src/main/java/com/evolveum/midpoint/repo/common/activity/state/CurrentActivityState.java b/.../src/main/java/com/evolveum/midpoint/repo/common/activity/state/CurrentActivityState.java
@@ -239,7 +239,7 @@ private void setRealizationStateWithoutCommit(ActivityRealizationStateType value
                 .item(getRealizationStateItemPath()).replace(value)
                 .asItemDelta();
         getTask().modify(itemDelta);
-        LOGGER.info("Setting realization state to {} for {}", value, activityExecution);
+        LOGGER.trace("Setting realization state to {} for {}", value, activityExecution);
     }
     //endregion
 
@@ -254,7 +254,7 @@ private void setResultStateNoCommit(OperationResultStatus resultStatus) throws S
                 .item(getResultStatusItemPath()).replace(OperationResultStatus.createStatusType(resultStatus))
                 .asItemDelta();
         getTask().modify(itemDelta);
-        LOGGER.info("Setting result status to {} for {}", resultStatus, activityExecution);
+        LOGGER.trace("Setting result status to {} for {}", resultStatus, activityExecution);
     }
     //endregion
 

diff --git a/...com/evolveum/midpoint/repo/common/activity/state/counters/CountersIncrementOperation.java b/...com/evolveum/midpoint/repo/common/activity/state/counters/CountersIncrementOperation.java
@@ -95,10 +95,9 @@ private void incrementCountersInRepository(OperationResult result)
                         .asItemDelta();
             }
             deltas.add(itemDelta);
-            System.out.println("New value for " + counterIdentifier + " is " + newValue); // TODO remove
             updatedValues.put(counterIdentifier, newValue);
         }
-        LOGGER.info("Counter deltas:\n{}", DebugUtil.debugDumpLazily(deltas, 1)); // TODO trace
+        LOGGER.trace("Counter deltas:\n{}", DebugUtil.debugDumpLazily(deltas, 1));
         return deltas;
     }