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

Posted to commits@lucene.apache.org by jp...@apache.org on 2015/06/19 15:20:47 UTC

svn commit: r1686408 - /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/search/BooleanQuery.java

Author: jpountz
Date: Fri Jun 19 13:20:47 2015
New Revision: 1686408

URL: http://svn.apache.org/r1686408
Log:
LUCENE-6570: Remove controversial defensive cloning of queries when adding them to a BooleanQuery.

Modified:
    lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/search/BooleanQuery.java

Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/search/BooleanQuery.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/search/BooleanQuery.java?rev=1686408&r1=1686407&r2=1686408&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/search/BooleanQuery.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/search/BooleanQuery.java Fri Jun 19 13:20:47 2015
@@ -122,7 +122,6 @@ public class BooleanQuery extends Query
       if (clauses.size() >= maxClauseCount) {
         throw new TooManyClauses();
       }
-      query = query.clone(); // be defensive
       clauses.add(new BooleanClause(query, occur));
       return this;
     }