Revert "Flag nested fields as non-aggregatable (#51774)"

This reverts commit c7046a0
elastic · Jan 14, 2020 · 01ca3cc · 01ca3cc
1 parent c622a2f
commit 01ca3cc
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 17 deletions.
diff --git a/...ins/data/server/index_patterns/fetcher/lib/field_capabilities/field_caps_response.test.js b/...ins/data/server/index_patterns/fetcher/lib/field_capabilities/field_caps_response.test.js
@@ -143,13 +143,6 @@ describe('index_patterns/field_capabilities/field_caps_response', () => {
         expect(child).toHaveProperty('subType', { nested: { path: 'nested_object_parent' } });
       });
 
-      it('returns nested sub-fields as non-aggregatable', () => {
-        const fields = readFieldCapsResponse(esResponse);
-        // Normally a keyword field would be aggregatable, but the fact that it is nested overrides that
-        const child = fields.find(f => f.name === 'nested_object_parent.child.keyword');
-        expect(child).toHaveProperty('aggregatable', false);
-      });
-
       it('handles fields that are both nested and multi', () => {
         const fields = readFieldCapsResponse(esResponse);
         const child = fields.find(f => f.name === 'nested_object_parent.child.keyword');

diff --git a/src/plugins/data/server/index_patterns/fetcher/lib/field_capabilities/field_caps_response.ts b/src/plugins/data/server/index_patterns/fetcher/lib/field_capabilities/field_caps_response.ts
@@ -182,14 +182,6 @@ export function readFieldCapsResponse(fieldCapsResponse: FieldCapsResponse): Fie
 
       if (Object.keys(subType).length > 0) {
         field.subType = subType;
-
-        // We don't support aggregating on nested fields, trying to do so in the UI will return
-        // blank results. For now we will stop showing nested fields as an option for aggregation.
-        // Once we add support for nested fields this condition should be removed and old index
-        // patterns should be migrated.
-        if (field.subType.nested) {
-          field.aggregatable = false;
-        }
       }
     }
   });

diff --git a/test/api_integration/apis/index_patterns/fields_for_wildcard_route/response.js b/test/api_integration/apis/index_patterns/fields_for_wildcard_route/response.js
@@ -72,7 +72,7 @@ export default function({ getService }) {
               readFromDocValues: true,
             },
             {
-              aggregatable: false,
+              aggregatable: true,
               esTypes: ['keyword'],
               name: 'nestedField.child',
               readFromDocValues: true,
@@ -154,7 +154,7 @@ export default function({ getService }) {
               readFromDocValues: true,
             },
             {
-              aggregatable: false,
+              aggregatable: true,
               esTypes: ['keyword'],
               name: 'nestedField.child',
               readFromDocValues: true,