Merge branch 'master' of github.com:elastic/kibana into metrics/log_e…

…vent_loop_delay
elastic · Jun 29, 2021 · c6e7e5c · c6e7e5c
2 parents 6ae5bc4 + 2e00e9c
commit c6e7e5c
Show file tree

Hide file tree

Showing 73 changed files with 2,014 additions and 329 deletions.
diff --git a/x-pack/plugins/lens/public/index.ts b/x-pack/plugins/lens/public/index.ts
@@ -54,6 +54,9 @@ export type {
   CounterRateIndexPatternColumn,
   DerivativeIndexPatternColumn,
   MovingAverageIndexPatternColumn,
+  FormulaIndexPatternColumn,
+  MathIndexPatternColumn,
+  OverallSumIndexPatternColumn,
 } from './indexpattern_datasource/types';
 export type { LensEmbeddableInput } from './editor_frame_service/embeddable';
 

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/loader.test.ts b/x-pack/plugins/lens/public/indexpattern_datasource/loader.test.ts
@@ -770,7 +770,7 @@ describe('loader', () => {
                 label: 'My hist',
                 operationType: 'date_histogram',
                 params: {
-                  interval: '1d',
+                  interval: 'm',
                 },
                 sourceField: 'timestamp',
               },

diff --git a/...lugins/lens/public/indexpattern_datasource/operations/definitions/date_histogram.test.tsx b/...lugins/lens/public/indexpattern_datasource/operations/definitions/date_histogram.test.tsx
@@ -176,30 +176,6 @@ describe('date_histogram', () => {
       });
       expect(column.params.interval).toEqual('auto');
     });
-
-    it('should create column object with restrictions', () => {
-      const column = dateHistogramOperation.buildColumn({
-        layer: { columns: {}, columnOrder: [], indexPatternId: '' },
-        indexPattern: createMockedIndexPattern(),
-        field: {
-          name: 'timestamp',
-          displayName: 'timestampLabel',
-          type: 'date',
-          esTypes: ['date'],
-          aggregatable: true,
-          searchable: true,
-          aggregationRestrictions: {
-            date_histogram: {
-              agg: 'date_histogram',
-              time_zone: 'UTC',
-              calendar_interval: '1y',
-            },
-          },
-        },
-      });
-      expect(column.params.interval).toEqual('1y');
-      expect(column.params.timeZone).toEqual('UTC');
-    });
   });
 
   describe('toEsAggsFn', () => {
@@ -223,7 +199,7 @@ describe('date_histogram', () => {
       );
     });
 
-    it('should not use normalized es interval for rollups', () => {
+    it('should use restricted time zone and omit use normalized es interval for rollups', () => {
       const esAggsFn = dateHistogramOperation.toEsAggsFn(
         layer.columns.col1 as DateHistogramIndexPatternColumn,
         'col1',
@@ -271,6 +247,7 @@ describe('date_histogram', () => {
           arguments: expect.objectContaining({
             interval: ['42w'],
             field: ['timestamp'],
+            time_zone: ['UTC'],
             useNormalizedEsInterval: [false],
           }),
         })
@@ -320,114 +297,6 @@ describe('date_histogram', () => {
     });
   });
 
-  describe('transfer', () => {
-    it('should adjust interval and time zone params if that is necessary due to restrictions', () => {
-      const transferedColumn = dateHistogramOperation.transfer!(
-        {
-          dataType: 'date',
-          isBucketed: true,
-          label: '',
-          operationType: 'date_histogram',
-          sourceField: 'dateField',
-          params: {
-            interval: 'd',
-          },
-        },
-        {
-          title: '',
-          id: '',
-          hasRestrictions: true,
-          fields: [
-            {
-              name: 'dateField',
-              displayName: 'dateField',
-              type: 'date',
-              aggregatable: true,
-              searchable: true,
-              aggregationRestrictions: {
-                date_histogram: {
-                  agg: 'date_histogram',
-                  time_zone: 'CET',
-                  calendar_interval: 'w',
-                },
-              },
-            },
-          ],
-          getFieldByName: getFieldByNameFactory([
-            {
-              name: 'dateField',
-              displayName: 'dateField',
-              type: 'date',
-              aggregatable: true,
-              searchable: true,
-              aggregationRestrictions: {
-                date_histogram: {
-                  agg: 'date_histogram',
-                  time_zone: 'CET',
-                  calendar_interval: 'w',
-                },
-              },
-            },
-          ]),
-        }
-      );
-      expect(transferedColumn).toEqual(
-        expect.objectContaining({
-          params: {
-            interval: 'w',
-            timeZone: 'CET',
-          },
-        })
-      );
-    });
-
-    it('should retain interval', () => {
-      const transferedColumn = dateHistogramOperation.transfer!(
-        {
-          dataType: 'date',
-          isBucketed: true,
-          label: '',
-          operationType: 'date_histogram',
-          sourceField: 'dateField',
-          params: {
-            interval: '20s',
-          },
-        },
-        {
-          title: '',
-          id: '',
-          hasRestrictions: false,
-          fields: [
-            {
-              name: 'dateField',
-              displayName: 'dateField',
-              type: 'date',
-              aggregatable: true,
-              searchable: true,
-            },
-          ],
-          getFieldByName: getFieldByNameFactory([
-            {
-              name: 'dateField',
-              displayName: 'dateField',
-              type: 'date',
-              aggregatable: true,
-              searchable: true,
-            },
-          ]),
-        }
-      );
-      expect(transferedColumn).toEqual(
-        expect.objectContaining({
-          params: {
-            interval: '20s',
-            timeZone: undefined,
-          },
-        })
-      );
-    });
-  });
-
   describe('param editor', () => {
     it('should render current value', () => {
       const updateLayerSpy = jest.fn();

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/date_histogram.tsx b/x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/date_histogram.tsx
@@ -45,7 +45,6 @@ export interface DateHistogramIndexPatternColumn extends FieldBasedIndexPatternC
   operationType: 'date_histogram';
   params: {
     interval: string;
-    timeZone?: string;
   };
 }
 
@@ -106,12 +105,6 @@ export const dateHistogramOperation: OperationDefinition<
   },
   getDefaultLabel: (column, indexPattern) => getSafeName(column.sourceField, indexPattern),
   buildColumn({ field }, columnParams) {
-    let interval = columnParams?.interval ?? autoInterval;
-    let timeZone: string | undefined;
-    if (field.aggregationRestrictions && field.aggregationRestrictions.date_histogram) {
-      interval = restrictedInterval(field.aggregationRestrictions) as string;
-      timeZone = field.aggregationRestrictions.date_histogram.time_zone;
-    }
     return {
       label: field.displayName,
       dataType: 'date',
@@ -120,8 +113,7 @@ export const dateHistogramOperation: OperationDefinition<
       isBucketed: true,
       scale: 'interval',
       params: {
-        interval,
-        timeZone,
+        interval: columnParams?.interval ?? autoInterval,
       },
     };
   },
@@ -135,28 +127,6 @@ export const dateHistogramOperation: OperationDefinition<
         (!newField.aggregationRestrictions || newField.aggregationRestrictions.date_histogram)
     );
   },
-  transfer: (column, newIndexPattern) => {
-    const newField = newIndexPattern.getFieldByName(column.sourceField);
-
-    if (newField?.aggregationRestrictions?.date_histogram) {
-      const restrictions = newField.aggregationRestrictions.date_histogram;
-
-      return {
-        ...column,
-        params: {
-          ...column.params,
-          timeZone: restrictions.time_zone,
-          // TODO this rewrite logic is simplified - if the current interval is a multiple of
-          // the restricted interval, we could carry it over directly. However as the current
-          // UI does not allow to select multiples of an interval anyway, this is not included yet.
-          // If the UI allows to pick more complicated intervals, this should be re-visited.
-          interval: restrictedInterval(newField.aggregationRestrictions) as string,
-        },
-      };
-    }
-
-    return column;
-  },
   onFieldChange: (oldColumn, field) => {
     return {
       ...oldColumn,
@@ -166,14 +136,24 @@ export const dateHistogramOperation: OperationDefinition<
   },
   toEsAggsFn: (column, columnId, indexPattern) => {
     const usedField = indexPattern.getFieldByName(column.sourceField);
+    let timeZone: string | undefined;
+    let interval = column.params?.interval ?? autoInterval;
+    if (
+      usedField &&
+      usedField.aggregationRestrictions &&
+      usedField.aggregationRestrictions.date_histogram
+    ) {
+      interval = restrictedInterval(usedField.aggregationRestrictions) as string;
+      timeZone = usedField.aggregationRestrictions.date_histogram.time_zone;
+    }
     return buildExpressionFunction<AggFunctionsMapping['aggDateHistogram']>('aggDateHistogram', {
       id: columnId,
       enabled: true,
       schema: 'segment',
       field: column.sourceField,
-      time_zone: column.params.timeZone,
+      time_zone: timeZone,
       useNormalizedEsInterval: !usedField?.aggregationRestrictions?.date_histogram,
-      interval: column.params.interval,
+      interval,
       drop_partials: false,
       min_doc_count: 0,
       extended_bounds: JSON.stringify({}),
@@ -244,7 +224,7 @@ export const dateHistogramOperation: OperationDefinition<
                 id="xpack.lens.indexPattern.dateHistogram.restrictedInterval"
                 defaultMessage="Interval fixed to {intervalValue} due to aggregation restrictions."
                 values={{
-                  intervalValue: currentColumn.params.interval,
+                  intervalValue: restrictedInterval(field!.aggregationRestrictions),
                 }}
               />
             ) : (

diff --git a/...s/public/indexpattern_datasource/operations/definitions/formula/editor/formula_editor.tsx b/...s/public/indexpattern_datasource/operations/definitions/formula/editor/formula_editor.tsx
@@ -24,7 +24,7 @@ import { monaco } from '@kbn/monaco';
 import classNames from 'classnames';
 import { CodeEditor } from '../../../../../../../../../src/plugins/kibana_react/public';
 import type { CodeEditorProps } from '../../../../../../../../../src/plugins/kibana_react/public';
-import { useDebounceWithOptions } from '../../../../../shared_components';
+import { TooltipWrapper, useDebounceWithOptions } from '../../../../../shared_components';
 import { ParamEditorProps } from '../../index';
 import { getManagedColumnsFrom } from '../../../layer_helpers';
 import { ErrorWrapper, runASTValidation, tryToParse } from '../validation';
@@ -729,11 +729,16 @@ export function FormulaEditor({
                       </EuiLink>
                     </EuiToolTip>
                   ) : (
-                    <EuiToolTip
-                      content={i18n.translate('xpack.lens.formula.editorHelpOverlayToolTip', {
-                        defaultMessage: 'Function reference',
-                      })}
+                    <TooltipWrapper
+                      tooltipContent={i18n.translate(
+                        'xpack.lens.formula.editorHelpOverlayToolTip',
+                        {
+                          defaultMessage: 'Function reference',
+                        }
+                      )}
+                      condition={!isHelpOpen}
                       position="top"
+                      delay="regular"
                     >
                       <EuiPopover
                         panelClassName="lnsFormula__docs lnsFormula__docs--overlay"
@@ -745,7 +750,12 @@ export function FormulaEditor({
                         button={
                           <EuiButtonIcon
                             className="lnsFormula__editorHelp lnsFormula__editorHelp--overlay"
-                            onClick={() => setIsHelpOpen(!isHelpOpen)}
+                            onClick={() => {
+                              if (!isHelpOpen) {
+                                trackUiEvent('open_formula_popover');
+                              }
+                              setIsHelpOpen(!isHelpOpen);
+                            }}
                             iconType="documentation"
                             color="text"
                             size="s"
@@ -764,7 +774,7 @@ export function FormulaEditor({
                           operationDefinitionMap={visibleOperationsMap}
                         />
                       </EuiPopover>
-                    </EuiToolTip>
+                    </TooltipWrapper>
                   )}
                 </EuiFlexItem>
 

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/index.ts b/x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/index.ts
@@ -118,6 +118,7 @@ export {
 export { CountIndexPatternColumn } from './count';
 export { LastValueIndexPatternColumn } from './last_value';
 export { RangeIndexPatternColumn } from './ranges';
+export { FormulaIndexPatternColumn, MathIndexPatternColumn } from './formula';
 
 // List of all operation definitions registered to this data source.
 // If you want to implement a new operation, add the definition to this array and

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/operations/index.ts b/x-pack/plugins/lens/public/indexpattern_datasource/operations/index.ts
@@ -35,4 +35,6 @@ export {
   OverallMinIndexPatternColumn,
   OverallMaxIndexPatternColumn,
   OverallAverageIndexPatternColumn,
+  FormulaIndexPatternColumn,
+  MathIndexPatternColumn,
 } from './definitions';
diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/operations/layer_helpers.test.ts b/x-pack/plugins/lens/public/indexpattern_datasource/operations/layer_helpers.test.ts
@@ -25,6 +25,7 @@ import { documentField } from '../document_field';
 import { getFieldByNameFactory } from '../pure_helpers';
 import { generateId } from '../../id_generator';
 import { createMockedFullReference, createMockedManagedReference } from './mocks';
+import { IndexPatternColumn, OperationDefinition } from './definitions';
 import { TinymathAST } from 'packages/kbn-tinymath';
 
 jest.mock('../operations');
@@ -2642,7 +2643,15 @@ describe('state_helpers', () => {
       });
     });
 
+    // no operation currently requires this check, faking a transfer function to check whether the generic logic works
     it('should rewrite column params if that is necessary due to restrictions', () => {
+      operationDefinitionMap.date_histogram.transfer = ((oldColumn) => ({
+        ...oldColumn,
+        params: {
+          ...oldColumn.params,
+          interval: 'w',
+        },
+      })) as OperationDefinition<IndexPatternColumn, 'field'>['transfer'];
       const layer: IndexPatternLayer = {
         columnOrder: ['col1', 'col2'],
         columns: {
@@ -2666,10 +2675,10 @@ describe('state_helpers', () => {
           ...layer.columns.col1,
           params: {
             interval: 'w',
-            timeZone: 'CET',
           },
         },
       });
+      delete operationDefinitionMap.date_histogram.transfer;
     });
 
     it('should remove operations referencing fields with wrong field types', () => {

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/types.ts b/x-pack/plugins/lens/public/indexpattern_datasource/types.ts
@@ -32,6 +32,9 @@ export {
   CounterRateIndexPatternColumn,
   DerivativeIndexPatternColumn,
   MovingAverageIndexPatternColumn,
+  FormulaIndexPatternColumn,
+  MathIndexPatternColumn,
+  OverallSumIndexPatternColumn,
 } from './operations';
 
 export type DraggedField = DragDropIdentifier & {