Skip to content

Commit 8ab0278

Browse files
authored
SOLR-13350, SOLR-17298: multi-threaded search: revert TestFiltering.java change (#2589)
1 parent c625d13 commit 8ab0278

File tree

1 file changed

+2
-16
lines changed

1 file changed

+2
-16
lines changed

solr/core/src/test/org/apache/solr/search/TestFiltering.java

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import java.util.Locale;
2323
import org.apache.lucene.search.DocIdSetIterator;
2424
import org.apache.lucene.search.Query;
25-
import org.apache.lucene.util.Bits;
2625
import org.apache.lucene.util.FixedBitSet;
2726
import org.apache.solr.SolrTestCaseJ4;
2827
import org.apache.solr.common.SolrInputDocument;
@@ -93,34 +92,21 @@ public void testLiveDocsSharing() throws Exception {
9392
QueryResult res = new QueryResult();
9493
searcher.search(res, cmd);
9594
set = res.getDocSet();
96-
assertEffectivelySame(set.getFixedBitSet(), live.getFixedBitSet());
95+
assertSame(set, live);
9796

9897
cmd.setQuery(QParser.getParser(qstr + " OR id:0", null, req).getQuery());
9998
cmd.setFilterList(QParser.getParser(qstr + " OR id:1", null, req).getQuery());
10099
res = new QueryResult();
101100
searcher.search(res, cmd);
102101
set = res.getDocSet();
103-
assertEffectivelySame(set.getFixedBitSet(), live.getFixedBitSet());
102+
assertSame(set, live);
104103
}
105104

106105
} finally {
107106
req.close();
108107
}
109108
}
110109

111-
/** If the a XOR b == 0, then both a & b are effectively the same bitset */
112-
private void assertEffectivelySame(FixedBitSet a, FixedBitSet b) {
113-
FixedBitSet xor = a.clone();
114-
xor.xor(b);
115-
assertEquals(new FixedBitSet(xor.length()), xor);
116-
}
117-
118-
private String bitsString(Bits bits) {
119-
StringBuilder s = new StringBuilder();
120-
for (int i = 0; i < bits.length(); i++) s.append(bits.get(i) ? 1 : 0);
121-
return s.toString();
122-
}
123-
124110
public void testCaching() throws Exception {
125111
clearIndex();
126112
assertU(adoc("id", "4", "val_i", "1"));

0 commit comments

Comments
 (0)