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 th...@apache.org on 2014/12/15 18:12:23 UTC

svn commit: r1643080 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/strategy/ContentMirrorStoreStrategy.java

Author: thomasm
Date: Wed Dec  3 10:29:28 2014
New Revision: 1643080

URL: http://svn.apache.org/r1643080
Log:
OAK-2315 Index cost is sometimes negative

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/strategy/ContentMirrorStoreStrategy.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/strategy/ContentMirrorStoreStrategy.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/strategy/ContentMirrorStoreStrategy.java?rev=1643080&r1=1643079&r2=1643080&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/strategy/ContentMirrorStoreStrategy.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/strategy/ContentMirrorStoreStrategy.java Wed Dec  3 10:29:28 2014
@@ -213,6 +213,7 @@ public class ContentMirrorStoreStrategy
                 // (entryCount could be Long.MAX_VALUE)
                 // the cost is not multiplied by the size, 
                 // otherwise the traversing index might be used
+                keyCount = Math.max(1, keyCount);
                 return (long) ((double) entryCount / keyCount) + size;
             }
             max = Math.max(10, max / size);