Allow runtime fields to depend on one another

This updates a test so that runtime fields depend on one another and fixes support for it.
elastic · Jul 15, 2020 · 70f7390 · 70f7390
1 parent 719ef1f
commit 70f7390
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/server/src/main/java/org/elasticsearch/index/query/QueryShardContext.java b/server/src/main/java/org/elasticsearch/index/query/QueryShardContext.java
@@ -297,8 +297,7 @@ MappedFieldType failIfFieldMappingNotFound(String name, MappedFieldType fieldMap
 
     public SearchLookup lookup() {
         if (lookup == null) {
-            lookup = new SearchLookup(getMapperService(),
-                    mappedFieldType -> indexFieldDataService.apply(mappedFieldType, fullyQualifiedIndex.getName()));
+            lookup = new SearchLookup(getMapperService(), this::getForField);
         }
         return lookup;
     }

diff --git a/x-pack/plugin/src/test/resources/rest-api-spec/test/runtime_fields/10_keyword.yml b/x-pack/plugin/src/test/resources/rest-api-spec/test/runtime_fields/10_keyword.yml
@@ -25,9 +25,10 @@ setup:
                 type: script
                 runtime_type: keyword
                 script: |
-                  String dow = doc['timestamp'].value.dayOfWeekEnum.getDisplayName(TextStyle.FULL, Locale.ROOT);
-                  if (dow.startsWith('T')) {
-                    value(dow);
+                  for (String dow: doc['day_of_week']) {
+                    if (dow.startsWith('T')) {
+                      value(dow);
+                    }
                   }
 
   - do: