svn commit: r1770937 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java

[thomasm]

Author: thomasm
Date: Wed Nov 23 09:31:23 2016
New Revision: 1770937

URL: http://svn.apache.org/viewvc?rev=1770937&view=rev
Log:
OAK-5145 Warn traversal queries: avoid repeated log entries

Modified:
    
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java

Modified: 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java?rev=1770937&r1=1770936&r2=1770937&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
 Wed Nov 23 09:31:23 2016
@@ -140,6 +140,8 @@ public class QueryImpl implements Query
     public static final String REP_SUGGEST = "rep:suggest()";
 
     private static final Logger LOG = LoggerFactory.getLogger(QueryImpl.class);
+    
+    private boolean potentiallySlowTraversalQueryLogged;
 
     private static final Ordering<QueryIndex> MINIMAL_COST_ORDERING = new 
Ordering<QueryIndex>() {
         @Override
@@ -1063,10 +1065,16 @@ public class QueryImpl implements Query
             case OK:
                 break;
             case WARN:
-                LOG.info(message);
+                if (!potentiallySlowTraversalQueryLogged) {
+                    LOG.info(message);
+                    potentiallySlowTraversalQueryLogged = true;
+                }
                 break;
             case FAIL:
-                LOG.warn(message);
+                if (!potentiallySlowTraversalQueryLogged) {
+                    LOG.warn(message);
+                    potentiallySlowTraversalQueryLogged = true;
+                }
                 throw new IllegalArgumentException(message);
             }
         }