Skip to content

Commit a6c154a

Browse files
committed
Use dummy TermStatistics when term is not found Closes #10660
1 parent 069e11b commit a6c154a

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

src/main/java/org/elasticsearch/action/termvectors/TermVectorsWriter.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,9 +104,14 @@ void setFields(Fields termVectorsByField, Set<String> selectedFields, EnumSet<Fl
104104
if (flags.contains(Flag.TermStatistics)) {
105105
// get the doc frequency
106106
if (dfs != null) {
107-
writeTermStatistics(dfs.termStatistics().get(term));
107+
final TermStatistics statistics = dfs.termStatistics().get(term);
108+
writeTermStatistics(statistics == null ? new TermStatistics(termBytesRef, 0, 0) : statistics);
108109
} else {
109-
writeTermStatistics(topLevelIterator);
110+
if (foundTerm) {
111+
writeTermStatistics(topLevelIterator);
112+
} else {
113+
writeTermStatistics(new TermStatistics(termBytesRef, 0, 0));
114+
}
110115
}
111116
}
112117
if (useDocsAndPos) {

src/test/java/org/elasticsearch/action/termvectors/GetTermVectorsTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -875,7 +875,7 @@ public void testArtificialNoDoc() throws IOException {
875875
checkBrownFoxTermVector(resp.getFields(), "field1", false);
876876
}
877877

878-
@Test @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/10660")
878+
@Test
879879
public void testArtificialNonExistingField() throws Exception {
880880
// setup indices
881881
ImmutableSettings.Builder settings = settingsBuilder()

0 commit comments

Comments
 (0)