fix: null check not required after elastic#106862

salvatore-campagna · Apr 2, 2024 · ab96ec1 · ab96ec1
1 parent 98d5c6a
commit ab96ec1
Showing 1 changed file with 0 additions and 6 deletions.
diff --git a/server/src/main/java/org/elasticsearch/search/fetch/subphase/FieldFetcher.java b/server/src/main/java/org/elasticsearch/search/fetch/subphase/FieldFetcher.java
@@ -73,12 +73,6 @@ public static FieldFetcher create(
 
             for (String field : context.getMatchingFieldNames(fieldPattern)) {
                 MappedFieldType ft = context.getFieldType(field);
-                // NOTE: fields that are not allowed for security reasons behave as if they are unmapped.
-                // In that case we do not expect to return values for those fields so the ValueFetcher
-                // is not needed.
-                if (ft == null) {
-                    continue;
-                }
                 if ("_size".equals(field) && includeSizeMetadataField == false) {
                     continue;
                 }