[unomi] branch master updated: UNOMI-708 Invalid "no matching query builder log" (#530)

[shuber]

This is an automated email from the ASF dual-hosted git repository.

shuber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/unomi.git


The following commit(s) were added to refs/heads/master by this push:
     new c036d347b UNOMI-708 Invalid "no matching query builder log" (#530)
c036d347b is described below

commit c036d347ba53156ac52cfdb6a31f6811f388772b
Author: Serge Huber <shu...@jahia.com>
AuthorDate: Tue Nov 1 11:50:51 2022 +0100

    UNOMI-708 Invalid "no matching query builder log" (#530)
    
    - Only display log if we really couldn't find a query builder and not 
anymore in the case where we couldn't match parameterized values.
---
 .../conditions/ConditionESQueryBuilderDispatcher.java        | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/persistence-elasticsearch/core/src/main/java/org/apache/unomi/persistence/elasticsearch/conditions/ConditionESQueryBuilderDispatcher.java
 
b/persistence-elasticsearch/core/src/main/java/org/apache/unomi/persistence/elasticsearch/conditions/ConditionESQueryBuilderDispatcher.java
index 97fb8109a..6bc8b5062 100644
--- 
a/persistence-elasticsearch/core/src/main/java/org/apache/unomi/persistence/elasticsearch/conditions/ConditionESQueryBuilderDispatcher.java
+++ 
b/persistence-elasticsearch/core/src/main/java/org/apache/unomi/persistence/elasticsearch/conditions/ConditionESQueryBuilderDispatcher.java
@@ -83,12 +83,12 @@ public class ConditionESQueryBuilderDispatcher {
             if (contextualCondition != null) {
                 return queryBuilder.buildQuery(contextualCondition, context, 
this);
             }
-        }
-
-        // if no matching
-        logger.warn("No matching query builder. See debug log level for more 
information");
-        if (logger.isDebugEnabled()) {
-            logger.debug("No matching query builder for condition {} and 
context {}", condition, context);
+        } else {
+            // if no matching
+            logger.warn("No matching query builder. See debug log level for 
more information");
+            if (logger.isDebugEnabled()) {
+                logger.debug("No matching query builder for condition {} and 
context {}", condition, context);
+            }
         }
 
         return QueryBuilders.matchAllQuery();