From 1e76f1a728a9796ac3123325c756bfa4a2243120 Mon Sep 17 00:00:00 2001 From: Ryan McKinley Date: Tue, 4 Jun 2019 10:30:07 +0200 Subject: [PATCH] DataSourceApi: remove ExploreDataSourceApi (#17424) --- packages/grafana-ui/src/types/datasource.ts | 16 +++++++++++----- public/app/features/explore/Explore.tsx | 4 ++-- public/app/features/explore/QueryRow.tsx | 4 ++-- .../loki/components/LokiQueryFieldForm.tsx | 4 ++-- .../prometheus/components/PromQueryField.tsx | 4 ++-- 5 files changed, 19 insertions(+), 13 deletions(-) diff --git a/packages/grafana-ui/src/types/datasource.ts b/packages/grafana-ui/src/types/datasource.ts index a2629ec6f6d..99354cd755e 100644 --- a/packages/grafana-ui/src/types/datasource.ts +++ b/packages/grafana-ui/src/types/datasource.ts @@ -202,14 +202,20 @@ export abstract class DataSourceApi< * Used by alerting to check if query contains template variables */ targetContainsTemplate?(query: TQuery): boolean; -} -export abstract class ExploreDataSourceApi< - TQuery extends DataQuery = DataQuery, - TOptions extends DataSourceJsonData = DataSourceJsonData -> extends DataSourceApi { + /** + * Used in explore + */ modifyQuery?(query: TQuery, action: QueryFixAction): TQuery; + + /** + * Used in explore + */ getHighlighterExpression?(query: TQuery): string[]; + + /** + * Used in explore + */ languageProvider?: any; } diff --git a/public/app/features/explore/Explore.tsx b/public/app/features/explore/Explore.tsx index 8028e8362d7..e5dbb508edc 100644 --- a/public/app/features/explore/Explore.tsx +++ b/public/app/features/explore/Explore.tsx @@ -31,7 +31,7 @@ import { } from './state/actions'; // Types -import { RawTimeRange, DataQuery, ExploreStartPageProps, ExploreDataSourceApi, DataQueryError } from '@grafana/ui'; +import { RawTimeRange, DataQuery, ExploreStartPageProps, DataSourceApi, DataQueryError } from '@grafana/ui'; import { ExploreItemState, ExploreUrlState, @@ -62,7 +62,7 @@ interface ExploreProps { changeSize: typeof changeSize; changeTime: typeof changeTime; datasourceError: string; - datasourceInstance: ExploreDataSourceApi; + datasourceInstance: DataSourceApi; datasourceLoading: boolean | null; datasourceMissing: boolean; exploreId: ExploreId; diff --git a/public/app/features/explore/QueryRow.tsx b/public/app/features/explore/QueryRow.tsx index 49880c11230..a348c2af451 100644 --- a/public/app/features/explore/QueryRow.tsx +++ b/public/app/features/explore/QueryRow.tsx @@ -16,7 +16,7 @@ import { StoreState } from 'app/types'; import { TimeRange, DataQuery, - ExploreDataSourceApi, + DataSourceApi, QueryFixAction, DataSourceStatus, PanelData, @@ -38,7 +38,7 @@ interface QueryRowProps extends PropsFromParent { changeQuery: typeof changeQuery; className?: string; exploreId: ExploreId; - datasourceInstance: ExploreDataSourceApi; + datasourceInstance: DataSourceApi; datasourceStatus: DataSourceStatus; highlightLogsExpressionAction: typeof highlightLogsExpressionAction; history: HistoryItem[]; diff --git a/public/app/plugins/datasource/loki/components/LokiQueryFieldForm.tsx b/public/app/plugins/datasource/loki/components/LokiQueryFieldForm.tsx index 210d2acb4cc..a23cd86ad58 100644 --- a/public/app/plugins/datasource/loki/components/LokiQueryFieldForm.tsx +++ b/public/app/plugins/datasource/loki/components/LokiQueryFieldForm.tsx @@ -16,7 +16,7 @@ import BracesPlugin from 'app/features/explore/slate-plugins/braces'; // Types import { LokiQuery } from '../types'; import { TypeaheadOutput, HistoryItem } from 'app/types/explore'; -import { ExploreDataSourceApi, ExploreQueryFieldProps, DataSourceStatus } from '@grafana/ui'; +import { DataSourceApi, ExploreQueryFieldProps, DataSourceStatus } from '@grafana/ui'; function getChooserText(hasSyntax: boolean, hasLogLabels: boolean, datasourceStatus: DataSourceStatus) { if (datasourceStatus === DataSourceStatus.Disconnected) { @@ -65,7 +65,7 @@ export interface CascaderOption { disabled?: boolean; } -export interface LokiQueryFieldFormProps extends ExploreQueryFieldProps, LokiQuery> { +export interface LokiQueryFieldFormProps extends ExploreQueryFieldProps, LokiQuery> { history: HistoryItem[]; syntax: any; logLabelOptions: any[]; diff --git a/public/app/plugins/datasource/prometheus/components/PromQueryField.tsx b/public/app/plugins/datasource/prometheus/components/PromQueryField.tsx index c432e9d58b4..d42447b6746 100644 --- a/public/app/plugins/datasource/prometheus/components/PromQueryField.tsx +++ b/public/app/plugins/datasource/prometheus/components/PromQueryField.tsx @@ -15,7 +15,7 @@ import BracesPlugin from 'app/features/explore/slate-plugins/braces'; import QueryField, { TypeaheadInput, QueryFieldState } from 'app/features/explore/QueryField'; import { PromQuery } from '../types'; import { CancelablePromise, makePromiseCancelable } from 'app/core/utils/CancelablePromise'; -import { ExploreDataSourceApi, ExploreQueryFieldProps, DataSourceStatus, QueryHint } from '@grafana/ui'; +import { DataSourceApi, ExploreQueryFieldProps, DataSourceStatus, QueryHint } from '@grafana/ui'; const HISTOGRAM_GROUP = '__histograms__'; const METRIC_MARK = 'metric'; @@ -101,7 +101,7 @@ interface CascaderOption { disabled?: boolean; } -interface PromQueryFieldProps extends ExploreQueryFieldProps, PromQuery> { +interface PromQueryFieldProps extends ExploreQueryFieldProps, PromQuery> { history: HistoryItem[]; }