This is an automated email from the ASF dual-hosted git repository. shuber pushed a commit to branch UNOMI-708-fix-wrong-log in repository https://gitbox.apache.org/repos/asf/unomi.git
commit 8d51829aa66e04cf8e3bc22871292ae761252d5b Author: Serge Huber <[email protected]> AuthorDate: Tue Nov 1 08:51:37 2022 +0100 UNOMI-708 Invalid "no matching query builder log" - 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();
