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 al...@apache.org on 2014/04/09 15:38:18 UTC

svn commit: r1585965 - /jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/FullTextSearchTest.java

Author: alexparvulescu
Date: Wed Apr  9 13:38:18 2014
New Revision: 1585965

URL: http://svn.apache.org/r1585965
Log:
OAK-1702 Create a benchmark for Full text search
 - temporarily removed expensive 'order by' clause

Modified:
    jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/FullTextSearchTest.java

Modified: jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/FullTextSearchTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/FullTextSearchTest.java?rev=1585965&r1=1585964&r2=1585965&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/FullTextSearchTest.java (original)
+++ jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/FullTextSearchTest.java Wed Apr  9 13:38:18 2014
@@ -88,7 +88,8 @@ public class FullTextSearchTest extends 
         @Override
         public Integer call() throws Exception {
             QueryManager qm = session.getWorkspace().getQueryManager();
-            Query q = qm.createQuery("/jcr:root//*[jcr:contains(@text, '" + word + "')] order by jcr:score()", Query.XPATH);
+            //TODO verify why "order by jcr:score()" accounts for what looks like > 20% of the perf lost in Collections.sort
+            Query q = qm.createQuery("/jcr:root//*[jcr:contains(@text, '" + word + "')] ", Query.XPATH);
             QueryResult r = q.execute();
             RowIterator it = r.getRows();
             checkArgument(it.hasNext(), "Not able to find entry with text [%s]", word);