You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ko...@apache.org on 2009/06/08 04:27:07 UTC

svn commit: r782506 - in /lucene/solr/trunk: CHANGES.txt src/java/org/apache/solr/search/DocSet.java

Author: koji
Date: Mon Jun  8 02:27:06 2009
New Revision: 782506

URL: http://svn.apache.org/viewvc?rev=782506&view=rev
Log:
SOLR-1207: equals method should compare this and other of DocList in DocSetBase

Modified:
    lucene/solr/trunk/CHANGES.txt
    lucene/solr/trunk/src/java/org/apache/solr/search/DocSet.java

Modified: lucene/solr/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/CHANGES.txt?rev=782506&r1=782505&r2=782506&view=diff
==============================================================================
--- lucene/solr/trunk/CHANGES.txt (original)
+++ lucene/solr/trunk/CHANGES.txt Mon Jun  8 02:27:06 2009
@@ -409,6 +409,8 @@
 48. SOLR-1182: Fix bug in OrdFieldSource#equals which could cause a bug with OrdFieldSource caching
                on OrdFieldSource#hashcode collisions. (Mark Miller)
 
+49. SOLR-1207: equals method should compare this and other of DocList in DocSetBase (koji)
+
 
 Other Changes
 ----------------------

Modified: lucene/solr/trunk/src/java/org/apache/solr/search/DocSet.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/search/DocSet.java?rev=782506&r1=782505&r2=782506&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/search/DocSet.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/search/DocSet.java Mon Jun  8 02:27:06 2009
@@ -162,7 +162,7 @@
     if (this instanceof DocList && other instanceof DocList) {
       // compare ordering
       DocIterator i1=this.iterator();
-      DocIterator i2=this.iterator();
+      DocIterator i2=other.iterator();
       while(i1.hasNext() && i2.hasNext()) {
         if (i1.nextDoc() != i2.nextDoc()) return false;
       }