honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix IndexerUtilsTests that relies on indexed fields 

This test creates doc values fields only but does not set the index options to none.
This commit fixes this discrepancy by adding an indexed point field for all doc values field.

relates elastic/x-pack-elasticsearch#4223

Original commit: elastic/x-pack-elasticsearch@8adab7c849
This commit is contained in:
Jim Ferenczi 2018-03-26 13:37:18 +02:00
parent 8c8de0a774
commit 3a75435980
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
plugin/rollup/src/test/java/org/elasticsearch/xpack/rollup/job/IndexerUtilsTests.java
View File

@ -6,6 +6,7 @@
package org.elasticsearch.xpack.rollup.job;
import org.apache.lucene.document.Document;
import org.apache.lucene.document.LongPoint;
import org.apache.lucene.document.SortedNumericDocValuesField;
import org.apache.lucene.index.DirectoryReader;
import org.apache.lucene.index.IndexReader;
@ -55,7 +56,6 @@ import static org.hamcrest.Matchers.equalTo;
import static org.mockito.Mockito.when;
public class IndexerUtilsTests extends AggregatorTestCase {
@AwaitsFix(bugUrl = "https://github.com/elastic/x-pack-elasticsearch/issues/4223")
public void testMissingFields() throws IOException {
String indexName = randomAlphaOfLengthBetween(1, 10);
RollupJobStats stats = new RollupJobStats(0, 0, 0, 0);
@ -71,6 +71,7 @@ public class IndexerUtilsTests extends AggregatorTestCase {
Document document = new Document();
long timestamp = new DateTime().minusDays(i).getMillis();
document.add(new SortedNumericDocValuesField(timestampField, timestamp));
document.add(new LongPoint(timestampField, timestamp));
document.add(new SortedNumericDocValuesField(valueField, randomIntBetween(1,100)));
indexWriter.addDocument(document);
}
@ -123,7 +124,6 @@ public class IndexerUtilsTests extends AggregatorTestCase {
}
}
@AwaitsFix(bugUrl = "https://github.com/elastic/x-pack-elasticsearch/issues/4223")
public void testCorrectFields() throws IOException {
String indexName = randomAlphaOfLengthBetween(1, 10);
RollupJobStats stats= new RollupJobStats(0, 0, 0, 0);
@ -139,6 +139,7 @@ public class IndexerUtilsTests extends AggregatorTestCase {
Document document = new Document();
long timestamp = new DateTime().minusDays(i).getMillis();
document.add(new SortedNumericDocValuesField(timestampField, timestamp));
document.add(new LongPoint(timestampField, timestamp));
document.add(new SortedNumericDocValuesField(valueField, randomIntBetween(1,100)));
indexWriter.addDocument(document);
}
@ -198,7 +199,6 @@ public class IndexerUtilsTests extends AggregatorTestCase {
}
}
@AwaitsFix(bugUrl = "https://github.com/elastic/x-pack-elasticsearch/issues/4223")
public void testNumericTerms() throws IOException {
String indexName = randomAlphaOfLengthBetween(1, 10);
RollupJobStats stats= new RollupJobStats(0, 0, 0, 0);
@ -213,6 +213,7 @@ public class IndexerUtilsTests extends AggregatorTestCase {
for (int i = 0; i < numDocs; i++) {
Document document = new Document();
document.add(new SortedNumericDocValuesField(valueField, i));
document.add(new LongPoint(valueField, i));
indexWriter.addDocument(document);
}
@ -262,7 +263,6 @@ public class IndexerUtilsTests extends AggregatorTestCase {
}
}
@AwaitsFix(bugUrl = "https://github.com/elastic/x-pack-elasticsearch/issues/4223")
public void testEmptyCounts() throws IOException {
String indexName = randomAlphaOfLengthBetween(1, 10);
RollupJobStats stats= new RollupJobStats(0, 0, 0, 0);
@ -278,6 +278,7 @@ public class IndexerUtilsTests extends AggregatorTestCase {
Document document = new Document();
long timestamp = new DateTime().minusDays(i).getMillis();
document.add(new SortedNumericDocValuesField(timestampField, timestamp));
document.add(new LongPoint(timestampField, timestamp));
document.add(new SortedNumericDocValuesField(valueField, randomIntBetween(1,100)));
indexWriter.addDocument(document);
}