commit 4bc9dc5d0a5dc7c60ab40be2134b72900601fd62
author David Pursehouse <dpursehouse@collab.net> Mon Aug 05 14:54:06 2019 +0900
committer David Pursehouse <dpursehouse@collab.net> Tue Aug 06 02:02:08 2019 +0000
tree d41855f904904a0e9cbdb520fc886bbfdf368202
parent aa85814fe9c86e282f4bb4241b1562ab83677778

PrologEnvironment: Reduce "setting reductionLimit" log spam

The log message "setting reductionLimit nnn" was emitted on every
call to PrologEnvironment#setPredicate, which in some cases can be
very frequent and thus spams the log.

We're only actually interested in the case where the reduction limit
is getting set to a value other than the default, which is only when
the predicate class is CONSULT_STREAM_2. Move the log statement into
the reductionLimit method, and only emit it in the case where the class
is CONSULT_STREAM_2.

Change-Id: Ief3a4f5fc3bc325cf21d0368d5c01fee0332a668

gerrit-server/src/main/java/com/google/gerrit/rules/PrologEnvironment.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/rules/PrologEnvironment.java b/gerrit-server/src/main/java/com/google/gerrit/rules/PrologEnvironment.java
index e98e480..f27e349 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/rules/PrologEnvironment.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/rules/PrologEnvironment.java
@@ -82,7 +82,6 @@
   public void setPredicate(Predicate goal) {
     super.setPredicate(goal);
     int reductionLimit = args.reductionLimit(goal);
-    log.debug("setting reductionLimit {}", reductionLimit);
     setReductionLimit(reductionLimit);
   }
 
@@ -212,6 +211,9 @@
 
     private int reductionLimit(Predicate goal) {
       if (goal.getClass() == CONSULT_STREAM_2) {
+        log.debug(
+            "predicate class is CONSULT_STREAM_2: override reductionLimit with compileLimit ({})",
+            compileLimit);
         return compileLimit;
       }
       return reductionLimit;