You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by ch...@apache.org on 2014/11/21 08:46:29 UTC

svn commit: r1640866 - /jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java

Author: chetanm
Date: Fri Nov 21 07:46:28 2014
New Revision: 1640866

URL: http://svn.apache.org/r1640866
Log:
OAK-2279 -  Add testMode support in IndexPlanner

Fix the testMode evaluation logic

Modified:
    jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java

Modified: jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java?rev=1640866&r1=1640865&r2=1640866&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java Fri Nov 21 07:46:28 2014
@@ -61,8 +61,12 @@ class IndexPlanner {
     IndexPlan getPlan() {
         IndexPlan.Builder builder = getPlanBuilder();
 
-        checkArgument(defn.isTestMode() && builder != null, "No plan found for filter [%s] " +
-                "while using definition [%s] and testMode is found to be enabled", filter, defn);
+        if (defn.isTestMode()
+                && builder == null){
+            String msg = String.format("No plan found for filter [%s] " +
+                    "while using definition [%s] and testMode is found to be enabled", filter, defn);
+            throw new IllegalStateException(msg);
+        }
 
         return builder != null ? builder.build() : null;
     }