[Step 1 ] VisEditors Telemetry enhancements (remove legacy agg-based telemetries)

src/plugins/telemetry/schema/oss_plugins.json

@@ -10028,122 +10028,6 @@
           }
         }
       }
-    },
-    "vis_type_table": {
-      "properties": {
-        "total": {
-          "type": "long"
-        },
-        "total_split": {
-          "type": "long"
-        },
-        "split_columns": {
-          "properties": {
-            "total": {
-              "type": "long"
-            },
-            "enabled": {
-              "type": "long"
-            }
-          }
-        },
-        "split_rows": {
-          "properties": {
-            "total": {
-              "type": "long"
-            },
-            "enabled": {
-              "type": "long"
-            }
-          }
-        }
-      }
-    },
-    "vis_type_timeseries": {
-      "properties": {
-        "timeseries_use_last_value_mode_total": {
-          "type": "long",
-          "_meta": {
-            "description": "Number of TSVB visualizations using \"last value\" as a time range"
-          }
-        },
-        "timeseries_use_es_indices_total": {
-          "type": "long",
-          "_meta": {
-            "description": "Number of TSVB visualizations using elasticsearch indices"
-          }
-        },
-        "timeseries_table_use_aggregate_function": {
-          "type": "long",
-          "_meta": {
-            "description": "Number of TSVB table visualizations using aggregate function"
-          }
-        },
-        "timeseries_types": {
-          "properties": {
-            "table": {
-              "type": "long"
-            },
-            "gauge": {
-              "type": "long"
-            },
-            "markdown": {
-              "type": "long"
-            },
-            "top_n": {
-              "type": "long"
-            },
-            "timeseries": {
-              "type": "long"
-            },
-            "metric": {
-              "type": "long"
-            }
-          }
-        }
-      }
-    },
-    "vis_type_vega": {
-      "properties": {
-        "vega_lib_specs_total": {
-          "type": "long"
-        },
-        "vega_lite_lib_specs_total": {
-          "type": "long"
-        },
-        "vega_use_map_total": {
-          "type": "long"
-        }
-      }
-    },
-    "visualization_types": {
-      "properties": {
-        "DYNAMIC_KEY": {
-          "properties": {
-            "total": {
-              "type": "long"
-            },
-            "spaces_min": {
-              "type": "long"
-            },
-            "spaces_max": {
-              "type": "long"
-            },
-            "spaces_avg": {
-              "type": "long"
-            },
-            "saved_7_days_total": {
-              "type": "long"
-            },
-            "saved_30_days_total": {
-              "type": "long"
-            },
-            "saved_90_days_total": {
-              "type": "long"
-            }
-          }
-        }
-      }
     }
   }
 }

src/plugins/vis_types/heatmap/public/editor/components/index.tsx

@@ -13,13 +13,12 @@ import { HeatmapVisParams, HeatmapTypeProps } from '../../types';
 const HeatmapOptionsLazy = lazy(() => import('./heatmap'));
 
 export const getHeatmapOptions =
-  ({ showElasticChartsOptions, palettes, trackUiMetric }: HeatmapTypeProps) =>
+  ({ showElasticChartsOptions, palettes }: HeatmapTypeProps) =>
   (props: VisEditorOptionsProps<HeatmapVisParams>) =>
     (
       <HeatmapOptionsLazy
         {...props}
         palettes={palettes}
         showElasticChartsOptions={showElasticChartsOptions}
-        trackUiMetric={trackUiMetric}
       />
     );

src/plugins/vis_types/heatmap/public/plugin.ts

@@ -34,16 +34,10 @@ export class VisTypeHeatmapPlugin {
     { visualizations, charts, usageCollection }: VisTypeHeatmapSetupDependencies
   ) {
     if (!core.uiSettings.get(LEGACY_HEATMAP_CHARTS_LIBRARY)) {
-      const trackUiMetric = usageCollection?.reportUiCounter.bind(
-        usageCollection,
-        'vis_type_heatmap'
-      );
-
       visualizations.createBaseVisualization(
         heatmapVisType({
           showElasticChartsOptions: true,
           palettes: charts.palettes,
-          trackUiMetric,
         })
       );
     }

src/plugins/vis_types/heatmap/public/types.ts

@@ -5,7 +5,6 @@
  * in compliance with, at your election, the Elastic License 2.0 or the Server
  * Side Public License, v 1.
  */
-import { UiCounterMetricType } from '@kbn/analytics';
 import type { Position } from '@elastic/charts';
 import type { ChartsPluginSetup, Style, Labels, ColorSchemas } from '@kbn/charts-plugin/public';
 import { Range } from '@kbn/expressions-plugin/public';
@@ -14,7 +13,6 @@ import { LegendSize } from '@kbn/visualizations-plugin/public';
 export interface HeatmapTypeProps {
   showElasticChartsOptions?: boolean;
   palettes?: ChartsPluginSetup['palettes'];
-  trackUiMetric?: (metricType: UiCounterMetricType, eventName: string | string[]) => void;
 }
 
 export interface HeatmapVisParams {

src/plugins/vis_types/heatmap/public/vis_type/heatmap.tsx

@@ -20,7 +20,6 @@ import { SplitTooltip } from './split_tooltip';
 export const getHeatmapVisTypeDefinition = ({
   showElasticChartsOptions = false,
   palettes,
-  trackUiMetric,
 }: HeatmapTypeProps): VisTypeDefinition<HeatmapVisParams> => ({
   name: 'heatmap',
   title: i18n.translate('visTypeHeatmap.heatmap.heatmapTitle', { defaultMessage: 'Heat map' }),
@@ -68,7 +67,6 @@ export const getHeatmapVisTypeDefinition = ({
     optionsTemplate: getHeatmapOptions({
       showElasticChartsOptions,
       palettes,
-      trackUiMetric,
     }),
     schemas: [
       {

src/plugins/vis_types/pie/public/editor/components/index.tsx

@@ -14,13 +14,12 @@ import { PieTypeProps } from '../../types';
 const PieOptionsLazy = lazy(() => import('./pie'));
 
 export const getPieOptions =
-  ({ showElasticChartsOptions, palettes, trackUiMetric }: PieTypeProps) =>
+  ({ showElasticChartsOptions, palettes }: PieTypeProps) =>
   (props: VisEditorOptionsProps<PartitionVisParams>) =>
     (
       <PieOptionsLazy
         {...props}
         palettes={palettes}
         showElasticChartsOptions={showElasticChartsOptions}
-        trackUiMetric={trackUiMetric}
       />
     );

src/plugins/vis_types/pie/public/editor/components/pie.tsx

@@ -7,7 +7,6 @@
  */
 
 import React, { useState, useEffect, useCallback } from 'react';
-import { METRIC_TYPE } from '@kbn/analytics';
 import {
   EuiPanel,
   EuiTitle,
@@ -223,9 +222,6 @@ const PieOptions = (props: PieOptionsProps) => {
               value={stateParams.nestedLegend}
               disabled={stateParams.legendDisplay === LegendDisplay.HIDE}
               setValue={(paramName, value) => {
-                if (props.trackUiMetric) {
-                  props.trackUiMetric(METRIC_TYPE.CLICK, 'nested_legend_switched');
-                }
                 setValue(paramName, value);
               }}
               data-test-subj="visTypePieNestedLegendSwitch"
@@ -253,9 +249,6 @@ const PieOptions = (props: PieOptionsProps) => {
             activePalette={stateParams.palette}
             paramName="palette"
             setPalette={(paramName, value) => {
-              if (props.trackUiMetric) {
-                props.trackUiMetric(METRIC_TYPE.CLICK, 'palette_selected');
-              }
               setValue(paramName, value);
             }}
           />
@@ -296,9 +289,6 @@ const PieOptions = (props: PieOptionsProps) => {
                 : stateParams.labels.position || LabelPositions.DEFAULT
             }
             setValue={(paramName, value) => {
-              if (props.trackUiMetric) {
-                props.trackUiMetric(METRIC_TYPE.CLICK, 'label_position_selected');
-              }
               setLabels(paramName, value);
             }}
             data-test-subj="visTypePieLabelPositionSelect"
@@ -338,9 +328,6 @@ const PieOptions = (props: PieOptionsProps) => {
               paramName="valuesFormat"
               value={stateParams.labels.valuesFormat || ValueFormats.PERCENT}
               setValue={(paramName, value) => {
-                if (props.trackUiMetric) {
-                  props.trackUiMetric(METRIC_TYPE.CLICK, 'values_format_selected');
-                }
                 setLabels(paramName, value);
               }}
               data-test-subj="visTypePieValueFormatsSelect"

src/plugins/vis_types/pie/public/plugin.ts

@@ -43,12 +43,10 @@ export class VisTypePiePlugin {
     { visualizations, charts, usageCollection }: VisTypePieSetupDependencies
   ) {
     if (!core.uiSettings.get(LEGACY_PIE_CHARTS_LIBRARY, false)) {
-      const trackUiMetric = usageCollection?.reportUiCounter.bind(usageCollection, 'vis_type_pie');
       visualizations.createBaseVisualization(
         pieVisType({
           showElasticChartsOptions: true,
           palettes: charts.palettes,
-          trackUiMetric,
         })
       );
     }

src/plugins/vis_types/pie/public/types/types.ts

@@ -6,7 +6,6 @@
  * Side Public License, v 1.
  */
 
-import { UiCounterMetricType } from '@kbn/analytics';
 import { SerializedFieldFormat } from '@kbn/field-formats-plugin/common';
 import { ChartsPluginSetup } from '@kbn/charts-plugin/public';
 
@@ -28,5 +27,4 @@ export interface Dimensions {
 export interface PieTypeProps {
   showElasticChartsOptions?: boolean;
   palettes?: ChartsPluginSetup['palettes'];
-  trackUiMetric?: (metricType: UiCounterMetricType, eventName: string | string[]) => void;
 }

src/plugins/vis_types/pie/public/vis_type/pie.ts

@@ -25,7 +25,6 @@ import { getPieOptions } from '../editor/components';
 export const getPieVisTypeDefinition = ({
   showElasticChartsOptions = false,
   palettes,
-  trackUiMetric,
 }: PieTypeProps): VisTypeDefinition<PartitionVisParams> => ({
   name: 'pie',
   title: i18n.translate('visTypePie.pie.pieTitle', { defaultMessage: 'Pie' }),
@@ -68,7 +67,6 @@ export const getPieVisTypeDefinition = ({
     optionsTemplate: getPieOptions({
       showElasticChartsOptions,
       palettes,
-      trackUiMetric,
     }),
     schemas: [
       {

src/plugins/vis_types/table/kibana.json

@@ -14,7 +14,6 @@
     "share",
     "visDefaultEditor"
   ],
-  "optionalPlugins": ["usageCollection"],
   "owner": {
     "name": "Vis Editors",
     "githubTeam": "kibana-vis-editors"

src/plugins/vis_types/table/server/index.ts

@@ -6,21 +6,14 @@
  * Side Public License, v 1.
  */
 
-import { CoreSetup, PluginConfigDescriptor } from '@kbn/core/server';
-import { UsageCollectionSetup } from '@kbn/usage-collection-plugin/server';
-
+import { PluginConfigDescriptor } from '@kbn/core/server';
 import { configSchema, ConfigSchema } from '../config';
-import { registerVisTypeTableUsageCollector } from './usage_collector';
 
 export const config: PluginConfigDescriptor<ConfigSchema> = {
   schema: configSchema,
 };
 
 export const plugin = () => ({
-  setup(core: CoreSetup, plugins: { usageCollection?: UsageCollectionSetup }) {
-    if (plugins.usageCollection) {
-      registerVisTypeTableUsageCollector(plugins.usageCollection);
-    }
-  },
+  setup() {},
   start() {},
 });