commit 19fb35ef4230d68ee2fa3834eba9d986f4f5d1a4
author Edwin Kempin <ekempin@google.com> Fri Jul 16 15:25:40 2021 +0200
committer Edwin Kempin <ekempin@google.com> Fri Jul 16 15:25:40 2021 +0200
tree fcc537d152e12db3c08013b187dfb4980b2ba292
parent 9628e43e8cf342ecb09e13d9c9a4168565622fe4

PrologRuleEvaluator: Add debug logs for input and output

Prolog rules can remove approvals. If that happens we want to be able to
see this in traces.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I5b0670b6985eb5d6a8685db3b415ff0d791c49a5

java/com/google/gerrit/server/rules/PrologRuleEvaluator.java

diff --git a/java/com/google/gerrit/server/rules/PrologRuleEvaluator.java b/java/com/google/gerrit/server/rules/PrologRuleEvaluator.java
index 3dc5be0..179a3d0 100644
--- a/java/com/google/gerrit/server/rules/PrologRuleEvaluator.java
+++ b/java/com/google/gerrit/server/rules/PrologRuleEvaluator.java
@@ -158,6 +158,8 @@
       return ruleError("Error looking up change " + cd.getId(), e);
     }
 
+    logger.atFine().log("input approvals: %s", cd.approvals());
+
     List<Term> results;
     try {
       results =
@@ -178,7 +180,9 @@
               getSubmitRuleName(), cd.getId(), projectState.getName()));
     }
 
-    return resultsToSubmitRecord(getSubmitRule(), results);
+    SubmitRecord submitRecord = resultsToSubmitRecord(getSubmitRule(), results);
+    logger.atFine().log("submit record: %s", submitRecord);
+    return submitRecord;
   }
 
   private String getSubmitRuleName() {
@@ -320,6 +324,7 @@
       logger.atSevere().withCause(e).log(err);
       return createRuleError(DEFAULT_MSG);
     }
+    logger.atFine().log("rule error: %s", err);
     return createRuleError(err);
   }