Skip to content

Commit 11ce6b9

Browse files
committed
test: Do not use random index writer as test expects a single segment
check against right version
1 parent 0266479 commit 11ce6b9

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

modules/percolator/src/main/java/org/elasticsearch/percolator/PercolateQueryBuilder.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@
4949
import org.elasticsearch.common.ParseField;
5050
import org.elasticsearch.common.ParsingException;
5151
import org.elasticsearch.common.bytes.BytesReference;
52-
import org.elasticsearch.common.io.Streams;
5352
import org.elasticsearch.common.io.stream.InputStreamStreamInput;
5453
import org.elasticsearch.common.io.stream.NamedWriteableAwareStreamInput;
5554
import org.elasticsearch.common.io.stream.NamedWriteableRegistry;
@@ -66,7 +65,6 @@
6665
import org.elasticsearch.index.analysis.FieldNameAnalyzer;
6766
import org.elasticsearch.index.fielddata.IndexFieldData;
6867
import org.elasticsearch.index.fielddata.IndexFieldDataCache;
69-
import org.elasticsearch.index.fielddata.SortedBinaryDocValues;
7068
import org.elasticsearch.index.mapper.DocumentMapper;
7169
import org.elasticsearch.index.mapper.DocumentMapperForType;
7270
import org.elasticsearch.index.mapper.MappedFieldType;
@@ -593,7 +591,7 @@ static PercolateQuery.QueryStore createStore(MappedFieldType queryBuilderFieldTy
593591
if (binaryDocValues == null) {
594592
return docId -> null;
595593
}
596-
if (indexVersion.onOrAfter(Version.V_6_0_0_beta1)) {
594+
if (indexVersion.onOrAfter(Version.V_6_1_0)) {
597595
return docId -> {
598596
if (binaryDocValues.advanceExact(docId)) {
599597
BytesRef qbSource = binaryDocValues.binaryValue();

modules/percolator/src/test/java/org/elasticsearch/percolator/QueryBuilderStoreTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,10 @@
2121
import org.apache.lucene.analysis.core.WhitespaceAnalyzer;
2222
import org.apache.lucene.index.DirectoryReader;
2323
import org.apache.lucene.index.IndexReader;
24+
import org.apache.lucene.index.IndexWriter;
2425
import org.apache.lucene.index.IndexWriterConfig;
2526
import org.apache.lucene.index.LeafReaderContext;
2627
import org.apache.lucene.index.NoMergePolicy;
27-
import org.apache.lucene.index.RandomIndexWriter;
2828
import org.apache.lucene.search.Query;
2929
import org.apache.lucene.search.TermQuery;
3030
import org.apache.lucene.store.Directory;
@@ -75,7 +75,7 @@ public void testStoringQueryBuilders() throws IOException {
7575
new Mapper.BuilderContext(settings, new ContentPath(0)));
7676

7777
Version version = randomBoolean() ? Version.V_5_6_0 : Version.V_6_1_0;
78-
try (RandomIndexWriter indexWriter = new RandomIndexWriter(random(), directory, config)) {
78+
try (IndexWriter indexWriter = new IndexWriter(directory, config)) {
7979
for (int i = 0; i < queryBuilders.length; i++) {
8080
queryBuilders[i] = new TermQueryBuilder(randomAlphaOfLength(4), randomAlphaOfLength(8));
8181
ParseContext parseContext = mock(ParseContext.class);

0 commit comments

Comments
 (0)