move query_utils to packages

x-pack/packages/ml/query_utils/index.ts

@@ -6,3 +6,5 @@
  */
 
 export { addExcludeFrozenToQuery } from './src/add_exclude_frozen_to_query';
+export { buildBaseFilterCriteria } from './src/build_base_filter_criteria';
+export { getSafeAggregationName } from './src/get_safe_aggregation_name';

x-pack/packages/ml/query_utils/src/add_exclude_frozen_to_query.test.ts

@@ -7,7 +7,7 @@
 
 import { addExcludeFrozenToQuery } from './add_exclude_frozen_to_query';
 
-describe('Util: addExcludeFrozenToQuery()', () => {
+describe('addExcludeFrozenToQuery', () => {
   test('Validation checks.', () => {
     expect(
       addExcludeFrozenToQuery({

x-pack/packages/ml/query_utils/src/build_base_filter_criteria.test.ts

@@ -0,0 +1,49 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+import { buildBaseFilterCriteria } from './build_base_filter_criteria';
+
+describe('buildBaseFilterCriteria', () => {
+  const earliestMs = 1483228800000; // 1 Jan 2017 00:00:00
+  const latestMs = 1485907199000; // 31 Jan 2017 23:59:59
+  const query = {
+    query_string: {
+      query: 'region:sa-east-1',
+      analyze_wildcard: true,
+      default_field: '*',
+    },
+  };
+
+  test('returns correct criteria for time range', () => {
+    expect(buildBaseFilterCriteria('timestamp', earliestMs, latestMs)).toEqual([
+      {
+        range: {
+          timestamp: {
+            gte: earliestMs,
+            lte: latestMs,
+            format: 'epoch_millis',
+          },
+        },
+      },
+    ]);
+  });
+
+  test('returns correct criteria for time range and query', () => {
+    expect(buildBaseFilterCriteria('timestamp', earliestMs, latestMs, query)).toEqual([
+      {
+        range: {
+          timestamp: {
+            gte: earliestMs,
+            lte: latestMs,
+            format: 'epoch_millis',
+          },
+        },
+      },
+      query,
+    ]);
+  });
+});

x-pack/plugins/ml/server/lib/query_utils.ts → x-pack/packages/ml/query_utils/src/build_base_filter_criteria.ts

@@ -5,19 +5,19 @@
  * 2.0.
  */
 
-/*
- * Contains utility functions for building and processing queries.
- */
+import type * as estypes from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
+import { Query } from '@kbn/es-query';
 
 // Builds the base filter criteria used in queries,
 // adding criteria for the time range and an optional query.
 export function buildBaseFilterCriteria(
   timeFieldName?: string,
   earliestMs?: number,
   latestMs?: number,
-  query?: object
-) {
+  query?: Query['query']
+): estypes.QueryDslQueryContainer[] {
   const filterCriteria = [];
+
   if (timeFieldName && earliestMs && latestMs) {
     filterCriteria.push({
       range: {
@@ -30,7 +30,7 @@ export function buildBaseFilterCriteria(
     });
   }
 
-  if (query) {
+  if (query && typeof query === 'object') {
     filterCriteria.push(query);
   }
 

x-pack/packages/ml/query_utils/src/get_safe_aggregation_name.test.ts

@@ -0,0 +1,20 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+import { getSafeAggregationName } from './get_safe_aggregation_name';
+
+describe('getSafeAggregationName', () => {
+  test('"foo" should be "foo"', () => {
+    expect(getSafeAggregationName('foo', 0)).toBe('foo');
+  });
+  test('"foo.bar" should be "foo.bar"', () => {
+    expect(getSafeAggregationName('foo.bar', 0)).toBe('foo.bar');
+  });
+  test('"foo&bar" should be "field_0"', () => {
+    expect(getSafeAggregationName('foo&bar', 0)).toBe('field_0');
+  });
+});

x-pack/packages/ml/query_utils/src/get_safe_aggregation_name.ts

@@ -0,0 +1,14 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+// Returns a name which is safe to use in elasticsearch aggregations for the supplied
+// field name. Aggregation names must be alpha-numeric and can only contain '_' and '-' characters,
+// so if the supplied field names contains disallowed characters, the provided index
+// identifier is used to return a safe 'dummy' name in the format 'field_index' e.g. field_0, field_1
+export function getSafeAggregationName(fieldName: string, index: number): string {
+  return fieldName.match(/^[a-zA-Z0-9-_.]+$/) ? fieldName : `field_${index}`;
+}

x-pack/packages/ml/query_utils/tsconfig.json

@@ -17,5 +17,6 @@
   ],
   "kbn_references": [
     "@kbn/ml-is-populated-object",
+    "@kbn/es-query",
   ]
 }

x-pack/plugins/aiops/public/application/utils/query_utils.test.ts → x-pack/plugins/aiops/public/application/utils/build_extended_base_filter_criteria.test.ts

@@ -9,7 +9,7 @@ import type { ChangePoint } from '@kbn/ml-agg-utils';
 
 import type { GroupTableItem } from '../../components/spike_analysis_table/types';
 
-import { buildBaseFilterCriteria } from './query_utils';
+import { buildExtendedBaseFilterCriteria } from './build_extended_base_filter_criteria';
 
 const selectedChangePointMock: ChangePoint = {
   doc_count: 53408,
@@ -41,9 +41,9 @@ const selectedGroupMock: GroupTableItem = {
 };
 
 describe('query_utils', () => {
-  describe('buildBaseFilterCriteria', () => {
+  describe('buildExtendedBaseFilterCriteria', () => {
     it('returns range filter based on minimum supplied arguments', () => {
-      const baseFilterCriteria = buildBaseFilterCriteria('the-time-field-name', 1234, 5678);
+      const baseFilterCriteria = buildExtendedBaseFilterCriteria('the-time-field-name', 1234, 5678);
 
       expect(baseFilterCriteria).toEqual([
         {
@@ -59,7 +59,7 @@ describe('query_utils', () => {
     });
 
     it('returns filters including default query with supplied arguments provided via UI', () => {
-      const baseFilterCriteria = buildBaseFilterCriteria(
+      const baseFilterCriteria = buildExtendedBaseFilterCriteria(
         '@timestamp',
         1640082000012,
         1640103600906,
@@ -81,7 +81,7 @@ describe('query_utils', () => {
     });
 
     it('includes a term filter when including a selectedChangePoint', () => {
-      const baseFilterCriteria = buildBaseFilterCriteria(
+      const baseFilterCriteria = buildExtendedBaseFilterCriteria(
         '@timestamp',
         1640082000012,
         1640103600906,
@@ -105,7 +105,7 @@ describe('query_utils', () => {
     });
 
     it('includes a term filter with must_not when excluding a selectedChangePoint', () => {
-      const baseFilterCriteria = buildBaseFilterCriteria(
+      const baseFilterCriteria = buildExtendedBaseFilterCriteria(
         '@timestamp',
         1640082000012,
         1640103600906,
@@ -130,7 +130,7 @@ describe('query_utils', () => {
     });
 
     it('includes multiple term filters when including a selectedGroupMock', () => {
-      const baseFilterCriteria = buildBaseFilterCriteria(
+      const baseFilterCriteria = buildExtendedBaseFilterCriteria(
         '@timestamp',
         1640082000012,
         1640103600906,
@@ -185,7 +185,7 @@ describe('query_utils', () => {
     });
 
     it('includes a must_not with nested term filters when excluding a selectedGroup', () => {
-      const baseFilterCriteria = buildBaseFilterCriteria(
+      const baseFilterCriteria = buildExtendedBaseFilterCriteria(
         '@timestamp',
         1640082000012,
         1640103600906,

x-pack/plugins/aiops/public/application/utils/query_utils.ts → x-pack/plugins/aiops/public/application/utils/build_extended_base_filter_criteria.ts

@@ -12,6 +12,9 @@ import type * as estypes from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
 
 import type { Query } from '@kbn/es-query';
 import type { ChangePoint, FieldValuePair } from '@kbn/ml-agg-utils';
+
+import { buildBaseFilterCriteria } from '@kbn/ml-query-utils';
+
 import type { GroupTableItem } from '../../components/spike_analysis_table/types';
 
 /*
@@ -20,7 +23,7 @@ import type { GroupTableItem } from '../../components/spike_analysis_table/types
 
 // Builds the base filter criteria used in queries,
 // adding criteria for the time range and an optional query.
-export function buildBaseFilterCriteria(
+export function buildExtendedBaseFilterCriteria(
   timeFieldName?: string,
   earliestMs?: number,
   latestMs?: number,
@@ -29,22 +32,7 @@ export function buildBaseFilterCriteria(
   includeSelectedChangePoint = true,
   selectedGroup?: GroupTableItem | null
 ): estypes.QueryDslQueryContainer[] {
-  const filterCriteria = [];
-  if (timeFieldName && earliestMs && latestMs) {
-    filterCriteria.push({
-      range: {
-        [timeFieldName]: {
-          gte: earliestMs,
-          lte: latestMs,
-          format: 'epoch_millis',
-        },
-      },
-    });
-  }
-
-  if (query && typeof query === 'object') {
-    filterCriteria.push(query);
-  }
+  const filterCriteria = buildBaseFilterCriteria(timeFieldName, earliestMs, latestMs, query);
 
   const groupFilter = [];
   if (selectedGroup) {

x-pack/plugins/aiops/public/get_document_stats.ts

@@ -13,7 +13,7 @@ import { isPopulatedObject } from '@kbn/ml-is-populated-object';
 import type { ChangePoint } from '@kbn/ml-agg-utils';
 import type { Query } from '@kbn/es-query';
 
-import { buildBaseFilterCriteria } from './application/utils/query_utils';
+import { buildExtendedBaseFilterCriteria } from './application/utils/build_extended_base_filter_criteria';
 import { GroupTableItem } from './components/spike_analysis_table/types';
 
 export interface DocumentCountStats {
@@ -54,7 +54,7 @@ export const getDocumentCountStatsRequest = (params: DocumentStatsSearchStrategy
   } = params;
 
   const size = 0;
-  const filterCriteria = buildBaseFilterCriteria(
+  const filterCriteria = buildExtendedBaseFilterCriteria(
     timeFieldName,
     earliestMs,
     latestMs,

x-pack/plugins/aiops/tsconfig.json

@@ -46,6 +46,7 @@
     "@kbn/ml-local-storage",
     "@kbn/ml-date-picker",
     "@kbn/ml-local-storage",
+    "@kbn/ml-query-utils",
   ],
   "exclude": [
     "target/**/*",

x-pack/plugins/data_visualizer/public/application/index_data_visualizer/hooks/use_field_stats.ts

@@ -12,6 +12,7 @@ import { last, cloneDeep } from 'lodash';
 import { mergeMap, switchMap } from 'rxjs/operators';
 import { Comparators } from '@elastic/eui';
 import type { ISearchOptions } from '@kbn/data-plugin/common';
+import { buildBaseFilterCriteria, getSafeAggregationName } from '@kbn/ml-query-utils';
 import type {
   DataStatsFetchProgress,
   FieldStatsSearchStrategyReturnBase,
@@ -22,10 +23,6 @@ import type {
 import { useDataVisualizerKibana } from '../../kibana_context';
 import type { FieldRequestConfig } from '../../../../common/types';
 import type { DataVisualizerIndexBasedAppState } from '../types/index_data_visualizer_state';
-import {
-  buildBaseFilterCriteria,
-  getSafeAggregationName,
-} from '../../../../common/utils/query_utils';
 import type { FieldStats, FieldStatsError } from '../../../../common/types/field_stats';
 import { getInitialProgress, getReducer } from '../progress_utils';
 import { MAX_EXAMPLES_DEFAULT } from '../search_strategy/requests/constants';

x-pack/plugins/data_visualizer/public/application/index_data_visualizer/search_strategy/requests/get_document_stats.ts

@@ -11,8 +11,8 @@ import { isPopulatedObject } from '@kbn/ml-is-populated-object';
 import { DataPublicPluginStart, ISearchOptions } from '@kbn/data-plugin/public';
 import seedrandom from 'seedrandom';
 import { isDefined } from '@kbn/ml-is-defined';
+import { buildBaseFilterCriteria } from '@kbn/ml-query-utils';
 import { RANDOM_SAMPLER_PROBABILITIES } from '../../constants/random_sampler';
-import { buildBaseFilterCriteria } from '../../../../../common/utils/query_utils';
 import type {
   DocumentCountStats,
   OverallStatsSearchStrategyParams,