chore(isthmus): debug log missing Calcite mapping (#516)

bestbeforetoday · web-flow · commit 5192e9f2a467 · 2025-09-19T16:44:25.000+01:00
There is an info level log in FunctionConverter for every every Calcite
operator for which there is no direct Substrait mapping. This message
makes the logs extremely busy and provides no value to end users. This
change reclassifies the message as debug log.

Signed-off-by: Mark S. Lewis &lt;Mark.S.Lewis@outlook.com&gt;
diff --git a/isthmus/src/main/java/io/substrait/isthmus/expression/FunctionConverter.java b/isthmus/src/main/java/io/substrait/isthmus/expression/FunctionConverter.java
@@ -90,7 +90,7 @@ public FunctionConverter(
     for (String key : alm.keySet()) {
       Collection<Sig> sigs = calciteOperators.get(key);
       if (sigs.isEmpty()) {
-        LOGGER.atInfo().log("No binding for function: {}", key);
+        LOGGER.atDebug().log("No binding for function: {}", key);
       }
 
       for (Sig sig : sigs) {

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ public FunctionConverter(`
`90`	`90`	`for (String key : alm.keySet()) {`
`91`	`91`	`Collection<Sig> sigs = calciteOperators.get(key);`
`92`	`92`	`if (sigs.isEmpty()) {`
`93`		`- LOGGER.atInfo().log("No binding for function: {}", key);`
	`93`	`+ LOGGER.atDebug().log("No binding for function: {}", key);`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`for (Sig sig : sigs) {`