[jackrabbit-oak] 01/01: OAK-10331 also log the caller of the query when no index is used

[jo...@apache.org]

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

joerghoh pushed a commit to branch OAK-10331
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git

commit efedc12cd12614c8c8ed8fea1898e8792421c5c0
Author: Joerg Hoh <jh...@adobe.com>
AuthorDate: Wed Jun 28 12:38:16 2023 +0200

    OAK-10331 also log the caller of the query when no index is used
---
 oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java b/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
index 51e6ead798..f210588bd7 100644
--- a/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
+++ b/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
@@ -36,6 +36,7 @@ import org.apache.jackrabbit.oak.api.Result.SizePrecision;
 import org.apache.jackrabbit.oak.api.Tree;
 import org.apache.jackrabbit.oak.namepath.JcrPathParser;
 import org.apache.jackrabbit.oak.namepath.NamePathMapper;
+import org.apache.jackrabbit.oak.plugins.index.IndexUtils;
 import org.apache.jackrabbit.oak.plugins.index.counter.jmx.NodeCounter;
 import org.apache.jackrabbit.oak.plugins.memory.PropertyValues;
 import org.apache.jackrabbit.oak.query.QueryOptions.Traversal;
@@ -1276,7 +1277,8 @@ public class QueryImpl implements Query {
                 // explicitly set in the query
                 traversal = queryOptions.traversal;
             }
-            String message = "Traversal query (query without index): " + statement + "; consider creating an index";
+            String caller = IndexUtils.getCaller(settings.getIgnoredClassNamesInCallTrace());
+            String message = "Traversal query (query without index): " + statement + "; called by " + caller + "; consider creating an index";
             switch (traversal) {
             case DEFAULT:
                 // not possible (changed to either FAIL or WARN above)