Add assert that doc values exist

parkertimmins · parkertimmins · commit 0fe4b4a2a638 · 2025-07-16T21:25:38.000-05:00
diff --git a/server/src/main/java/org/elasticsearch/index/mapper/KeywordFieldMapper.java b/server/src/main/java/org/elasticsearch/index/mapper/KeywordFieldMapper.java
@@ -495,6 +495,7 @@ private FieldType resolveFieldType(
         ) {
             if (requireDocValueSkippers) {
                 assert indexMode == IndexMode.LOGSDB;
+                assert hasDocValues.getValue();
                 return new FieldType(Defaults.FIELD_TYPE_WITH_SKIP_DOC_VALUES);
             }
             if (useDocValuesSkipper

Original file line number	Diff line number	Diff line change
`@@ -495,6 +495,7 @@ private FieldType resolveFieldType(`
`495`	`495`	`) {`
`496`	`496`	`if (requireDocValueSkippers) {`
`497`	`497`	`assert indexMode == IndexMode.LOGSDB;`
	`498`	`+ assert hasDocValues.getValue();`
`498`	`499`	`return new FieldType(Defaults.FIELD_TYPE_WITH_SKIP_DOC_VALUES);`
`499`	`500`	`}`
`500`	`501`	`if (useDocValuesSkipper`