From c02e6dce5df6bfb5e61319050456013d0b9d0f94 Mon Sep 17 00:00:00 2001
From: Daniil Suleiman <31325372+sulemanof@users.noreply.github.com>
Date: Thu, 2 Jul 2020 11:38:46 +0300
Subject: [PATCH] Filter out error when calculating a label (#69934)

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
---
 .../public/search/tabify/get_columns.test.ts     | 16 ++++++++++++++++
 .../data/public/search/tabify/get_columns.ts     |  9 ++++++++-
 2 files changed, 24 insertions(+), 1 deletion(-)

diff --git a/src/plugins/data/public/search/tabify/get_columns.test.ts b/src/plugins/data/public/search/tabify/get_columns.test.ts
index b4498aca4079c..07bf95ac51b3f 100644
--- a/src/plugins/data/public/search/tabify/get_columns.test.ts
+++ b/src/plugins/data/public/search/tabify/get_columns.test.ts
@@ -166,4 +166,20 @@ describe('get columns', () => {
       'Sum of @timestamp',
     ]);
   });
+
+  test('should not fail if there is no field for date histogram agg', () => {
+    const columns = tabifyGetColumns(
+      createAggConfigs([
+        {
+          type: 'date_histogram',
+          schema: 'segment',
+          params: {},
+        },
+        { type: 'sum', schema: 'metric', params: { field: '@timestamp' } },
+      ]).aggs,
+      false
+    );
+
+    expect(columns.map((c) => c.name)).toEqual(['', 'Sum of @timestamp']);
+  });
 });
diff --git a/src/plugins/data/public/search/tabify/get_columns.ts b/src/plugins/data/public/search/tabify/get_columns.ts
index 8c538288d2fea..8e907d4b0cb88 100644
--- a/src/plugins/data/public/search/tabify/get_columns.ts
+++ b/src/plugins/data/public/search/tabify/get_columns.ts
@@ -22,10 +22,17 @@ import { IAggConfig } from '../aggs';
 import { TabbedAggColumn } from './types';
 
 const getColumn = (agg: IAggConfig, i: number): TabbedAggColumn => {
+  let name = '';
+  try {
+    name = agg.makeLabel();
+  } catch (e) {
+    // skip the case when makeLabel throws an error (e.x. no appropriate field for an aggregation)
+  }
+
   return {
     aggConfig: agg,
     id: `col-${i}-${agg.id}`,
-    name: agg.makeLabel(),
+    name,
   };
 };