Skip to content

Commit 9377841

Browse files
refactor: merge query reducers (#1652)
1 parent ed9a03b commit 9377841

File tree

38 files changed

+263
-355
lines changed

38 files changed

+263
-355
lines changed

src/containers/Tenant/Diagnostics/TenantOverview/TenantCpu/TopQueries.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@ import React from 'react';
33
import {useHistory, useLocation} from 'react-router-dom';
44

55
import {parseQuery} from '../../../../../routes';
6-
import {changeUserInput} from '../../../../../store/reducers/executeQuery';
76
import {
87
setTopQueriesFilters,
98
topQueriesApi,
109
} from '../../../../../store/reducers/executeTopQueries/executeTopQueries';
10+
import {changeUserInput} from '../../../../../store/reducers/query/query';
1111
import {
1212
TENANT_DIAGNOSTICS_TABS_IDS,
1313
TENANT_PAGE,

src/containers/Tenant/Diagnostics/TopQueries/TopQueries.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@ import {DateRange} from '../../../../components/DateRange';
1111
import {Search} from '../../../../components/Search';
1212
import {TableWithControlsLayout} from '../../../../components/TableWithControlsLayout/TableWithControlsLayout';
1313
import {parseQuery} from '../../../../routes';
14-
import {changeUserInput} from '../../../../store/reducers/executeQuery';
1514
import {setTopQueriesFilters} from '../../../../store/reducers/executeTopQueries/executeTopQueries';
15+
import {changeUserInput} from '../../../../store/reducers/query/query';
1616
import {
1717
TENANT_PAGE,
1818
TENANT_PAGES_IDS,

src/containers/Tenant/ObjectSummary/SchemaTree/SchemaTree.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import React from 'react';
66
import {NavigationTree} from 'ydb-ui-components';
77

88
import {useCreateDirectoryFeatureAvailable} from '../../../../store/reducers/capabilities/hooks';
9-
import {selectUserInput} from '../../../../store/reducers/executeQuery';
9+
import {selectUserInput} from '../../../../store/reducers/query/query';
1010
import {schemaApi} from '../../../../store/reducers/schema/schema';
1111
import {tableSchemaDataApi} from '../../../../store/reducers/tableSchemaData';
1212
import type {GetTableSchemaDataParams} from '../../../../store/reducers/tableSchemaData';

src/containers/Tenant/Query/ExecuteResult/ExecuteResult.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@ import {LoaderWrapper} from '../../../../components/LoaderWrapper/LoaderWrapper'
1313
import {QueryExecutionStatus} from '../../../../components/QueryExecutionStatus';
1414
import {QueryResultTable} from '../../../../components/QueryResultTable/QueryResultTable';
1515
import {disableFullscreen} from '../../../../store/reducers/fullscreen';
16+
import type {QueryResult} from '../../../../store/reducers/query/types';
1617
import type {TKqpStatsQuery} from '../../../../types/api/query';
1718
import type {ValueOf} from '../../../../types/common';
18-
import type {ExecuteQueryResult} from '../../../../types/store/executeQuery';
1919
import {getArray} from '../../../../utils';
2020
import {cn} from '../../../../utils/cn';
2121
import {USE_SHOW_PLAN_SVG_KEY} from '../../../../utils/constants';
@@ -50,7 +50,7 @@ const resultOptionsIds = {
5050
type SectionID = ValueOf<typeof resultOptionsIds>;
5151

5252
interface ExecuteResultProps {
53-
result: ExecuteQueryResult;
53+
result: QueryResult;
5454
isResultsCollapsed?: boolean;
5555
theme?: string;
5656
tenantName: string;

src/containers/Tenant/Query/ExecuteResult/utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import {explainVersions} from '../../../../store/reducers/explainQuery/utils';
1+
import {explainVersions} from '../../../../store/reducers/query/prepareQueryData';
22
import type {IQueryResult} from '../../../../types/store/query';
33
import {preparePlan, prepareSimplifiedPlan} from '../../../../utils/prepareQueryExplain';
44
import {parseQueryExplainPlan} from '../../../../utils/query';

src/containers/Tenant/Query/ExplainResult/ExplainResult.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ import Fullscreen from '../../../../components/Fullscreen/Fullscreen';
99
import {LoaderWrapper} from '../../../../components/LoaderWrapper/LoaderWrapper';
1010
import {QueryExecutionStatus} from '../../../../components/QueryExecutionStatus';
1111
import {disableFullscreen} from '../../../../store/reducers/fullscreen';
12+
import type {QueryResult} from '../../../../store/reducers/query/types';
1213
import type {ValueOf} from '../../../../types/common';
13-
import type {ExplainQueryResult} from '../../../../types/store/executeQuery';
1414
import {cn} from '../../../../utils/cn';
1515
import {getStringifiedData} from '../../../../utils/dataFormatters/dataFormatters';
1616
import {useTypedDispatch} from '../../../../utils/hooks';
@@ -59,7 +59,7 @@ const explainOptions = [
5959

6060
interface ExplainResultProps {
6161
theme: string;
62-
result: ExplainQueryResult;
62+
result: QueryResult;
6363
tenantName: string;
6464
isResultsCollapsed?: boolean;
6565
onCollapseResults: VoidFunction;
@@ -81,7 +81,7 @@ export function ExplainResult({
8181
const [isPending, startTransition] = React.useTransition();
8282
const {error, isLoading, queryId} = result;
8383

84-
const {plan: explain, ast, simplifiedPlan} = result.data || {};
84+
const {preparedPlan: explain, ast, simplifiedPlan} = result.data || {};
8585

8686
React.useEffect(() => {
8787
return () => {

src/containers/Tenant/Query/ExplainResult/components/Graph/Graph.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import {YDBGraph} from '../../../../../../components/Graph/Graph';
2-
import type {PreparedExplainResponse} from '../../../../../../store/reducers/explainQuery/types';
3-
import {explainVersions} from '../../../../../../store/reducers/explainQuery/utils';
2+
import {explainVersions} from '../../../../../../store/reducers/query/prepareQueryData';
3+
import type {PreparedPlan} from '../../../../../../store/reducers/query/types';
44
import {cn} from '../../../../../../utils/cn';
55
import i18n from '../../i18n';
66

@@ -9,7 +9,7 @@ import './Graph.scss';
99
const b = cn('ydb-query-explain-graph');
1010

1111
interface GraphProps {
12-
explain: PreparedExplainResponse['plan'];
12+
explain: PreparedPlan;
1313
theme: string;
1414
}
1515

src/containers/Tenant/Query/ExplainResult/components/SimplifiedPlan/SimplifiedPlan.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import React from 'react';
33
import {Table, useTable} from '@gravity-ui/table';
44
import type {CellContext, ColumnDef, ExpandedState} from '@tanstack/react-table';
55

6-
import type {SimplifiedPlanItem} from '../../../../../../store/reducers/explainQuery/types';
6+
import type {SimplifiedPlanItem} from '../../../../../../store/reducers/query/types';
77
import {configuredNumeral} from '../../../../../../utils/numeral';
88
import {formatToMs} from '../../../../../../utils/timeParsers';
99
import {toExponential} from '../../../../../../utils/utils';

src/containers/Tenant/Query/ExplainResult/components/SimplifiedPlan/__tests__/utils.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import type {SimplifiedPlanItem} from '../../../../../../../store/reducers/explainQuery/types';
1+
import type {SimplifiedPlanItem} from '../../../../../../../store/reducers/query/types';
22
import type {ExtendedSimplifiesPlanItem} from '../types';
33
import {getExtendedTreeNodes} from '../utils';
44

src/containers/Tenant/Query/ExplainResult/components/SimplifiedPlan/types.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import type {SimplifiedPlanItem} from '../../../../../../store/reducers/explainQuery/types';
1+
import type {SimplifiedPlanItem} from '../../../../../../store/reducers/query/types';
22

33
export interface ExtendedSimplifiesPlanItem extends Omit<SimplifiedPlanItem, 'children'> {
44
lines?: string;

0 commit comments

Comments
 (0)