Skip to content

Commit

Permalink
cluster-ui: alphabetize imports
Browse files Browse the repository at this point in the history
This commit modifies the imports/order eslint rule to also
enforce that imports are alphabetically ordered by import path.

Epic: none

Release note: None
  • Loading branch information
xinhaoz committed Jul 23, 2024
1 parent dda59d4 commit c5f7261
Show file tree
Hide file tree
Showing 328 changed files with 1,420 additions and 1,417 deletions.
4 changes: 4 additions & 0 deletions pkg/ui/workspaces/cluster-ui/.eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,10 @@
"import/order": [
"error",
{
"alphabetize": {
"order": "asc",
"caseInsensitive": true
},
"newlines-between": "always",
"groups": [
"builtin",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import React, { FunctionComponent } from "react";
import classnames from "classnames/bind";
import { Badge, BadgeIntent, FuzzyTime } from "@cockroachlabs/ui-components";
import classnames from "classnames/bind";
import React, { FunctionComponent } from "react";

import {
NotificationTypeProp,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@

// This is a placeholder for real implementation (likely in Redux?) of notifications

import { notificationTypes, NotificationProps } from "../Notifications";
import { NotificationMessageProps } from "../NotificationMessage";
import { notificationTypes, NotificationProps } from "../Notifications";

export const generateNotificationProps = (
notifications: Array<NotificationProps>,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,18 @@
// licenses/APL.txt.

import * as protos from "@cockroachlabs/crdb-protobuf-client";
import moment from "moment-timezone";
import Long from "long";
import moment from "moment-timezone";

import { TimestampToMoment } from "../util";

import {
getActiveExecutionsFromSessions,
getAppsFromActiveExecutions,
filterActiveStatements,
filterActiveTransactions,
INTERNAL_APP_NAME_PREFIX,
} from "./activeStatementUtils";
import {
ActiveStatementPhase,
SessionsResponse,
Expand All @@ -24,13 +31,6 @@ import {
ActiveTransactionFilters,
ExecutionStatus,
} from "./types";
import {
getActiveExecutionsFromSessions,
getAppsFromActiveExecutions,
filterActiveStatements,
filterActiveTransactions,
INTERNAL_APP_NAME_PREFIX,
} from "./activeStatementUtils";

type ActiveQuery = protos.cockroach.server.serverpb.ActiveQuery;
const Timestamp = protos.google.protobuf.Timestamp;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@

import moment from "moment-timezone";

import { ClusterLocksResponse, ClusterLockState } from "src/api";
import { byteArrayToUuid } from "src/sessions";
import { TimestampToMoment, unset } from "src/util";
import { ClusterLocksResponse, ClusterLockState } from "src/api";
import { DurationToMomentDuration } from "src/util/convert";

import {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import React, { useMemo } from "react";
import classNames from "classnames/bind";
import React, { useMemo } from "react";

import {
ActiveStatement,
Expand All @@ -18,12 +18,12 @@ import {
import ColumnsSelector, {
SelectOption,
} from "src/columnsSelector/columnsSelector";
import { isSelectedColumn } from "src/columnsSelector/utils";
import { calculateActiveFilters } from "src/queryFilter/filter";
import sortableTableStyles from "src/sortedtable/sortedtable.module.scss";
import { EmptyStatementsPlaceholder } from "src/statementsPage/emptyStatementsPlaceholder";
import { TableStatistics } from "src/tableStatistics";
import { StatementViewType } from "src/statementsPage/statementPageTypes";
import { calculateActiveFilters } from "src/queryFilter/filter";
import { isSelectedColumn } from "src/columnsSelector/utils";
import { TableStatistics } from "src/tableStatistics";

import {
ISortedTablePagination,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,20 +8,20 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import { Tooltip } from "@cockroachlabs/ui-components";
import React from "react";
import { Link } from "react-router-dom";
import { Tooltip } from "@cockroachlabs/ui-components";

import { isSelectedColumn } from "../../columnsSelector/utils";
import { ColumnDescriptor } from "../../sortedtable";
import { limitText } from "../../util";
import {
activeStatementColumnsFromCommon,
ExecutionsColumn,
executionsTableTitles,
getLabel,
} from "../execTableCommon";
import { ActiveStatement } from "../types";
import { limitText } from "../../util";

export function makeActiveStatementsColumns(
isTenant: boolean,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import React, { useMemo } from "react";
import classNames from "classnames/bind";
import React, { useMemo } from "react";

import {
ActiveTransaction,
Expand All @@ -18,13 +18,13 @@ import {
import ColumnsSelector, {
SelectOption,
} from "src/columnsSelector/columnsSelector";
import { isSelectedColumn } from "src/columnsSelector/utils";
import { calculateActiveFilters } from "src/queryFilter/filter";
import { SortedTable } from "src/sortedtable";
import sortableTableStyles from "src/sortedtable/sortedtable.module.scss";
import { EmptyTransactionsPlaceholder } from "src/transactionsPage/emptyTransactionsPlaceholder";
import { TableStatistics } from "src/tableStatistics";
import { EmptyTransactionsPlaceholder } from "src/transactionsPage/emptyTransactionsPlaceholder";
import { TransactionViewType } from "src/transactionsPage/transactionsPageTypes";
import { calculateActiveFilters } from "src/queryFilter/filter";
import { isSelectedColumn } from "src/columnsSelector/utils";
import { SortedTable } from "src/sortedtable";

import {
ISortedTablePagination,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,20 +8,20 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import { Tooltip } from "@cockroachlabs/ui-components";
import React from "react";
import { Link } from "react-router-dom";
import { Tooltip } from "@cockroachlabs/ui-components";

import { isSelectedColumn } from "../../columnsSelector/utils";
import { ColumnDescriptor } from "../../sortedtable";
import { limitText } from "../../util";
import {
activeTransactionColumnsFromCommon,
ExecutionsColumn,
executionsTableTitles,
getLabel,
} from "../execTableCommon";
import { ActiveTransaction, ExecutionType } from "../types";
import { limitText } from "../../util";

export function makeActiveTransactionsColumns(
isTenant: boolean,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,16 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import { Tooltip } from "@cockroachlabs/ui-components";
import React from "react";
import { Link } from "react-router-dom";
import { Tooltip } from "@cockroachlabs/ui-components";

import { ColumnDescriptor, SortedTable } from "../../sortedtable";
import { ContendedExecution, ExecutionType } from "../types";
import { StatusIcon } from "../statusIcon";
import { executionsTableTitles } from "../execTableCommon";
import { DATE_FORMAT_24_TZ, Duration, limitText } from "../../util";
import { Timestamp } from "../../timestamp";
import { DATE_FORMAT_24_TZ, Duration, limitText } from "../../util";
import { executionsTableTitles } from "../execTableCommon";
import { StatusIcon } from "../statusIcon";
import { ContendedExecution, ExecutionType } from "../types";

const getID = (item: ContendedExecution, execType: ExecutionType) =>
execType === "transaction"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import React from "react";
import { Tooltip } from "@cockroachlabs/ui-components";
import React from "react";
import { Link } from "react-router-dom";

import { ColumnDescriptor } from "src/sortedtable";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@
// licenses/APL.txt.

import { Switch } from "antd";
import React from "react";
import classNames from "classnames/bind";
import { Moment } from "moment-timezone";
import React from "react";

import RefreshIcon from "src/icon/refreshIcon";
import { Timestamp } from "src/timestamp";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import React from "react";
import classNames from "classnames/bind";
import React from "react";

import { CircleFilled } from "src/icon";

import { ExecutionStatus } from "./types";
import styles from "./executionStatusIcon.module.scss";
import { ExecutionStatus } from "./types";

const cx = classNames.bind(styles);

Expand Down
2 changes: 1 addition & 1 deletion pkg/ui/workspaces/cluster-ui/src/anchor/anchor.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import React from "react";
import classnames from "classnames/bind";
import React from "react";

import styles from "./anchor.module.scss";

Expand Down
2 changes: 1 addition & 1 deletion pkg/ui/workspaces/cluster-ui/src/api/contentionApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,12 @@ import {
sqlResultsAreEmpty,
} from "./sqlApi";
import { TxnInsightDetailsRequest } from "./txnInsightDetailsApi";
import { makeInsightsSqlRequest } from "./txnInsightsUtils";
import {
FingerprintStmtsResponseColumns,
TxnStmtFingerprintsResponseColumns,
TxnWithStmtFingerprints,
} from "./txnInsightsApi";
import { makeInsightsSqlRequest } from "./txnInsightsUtils";

export type ContentionFilters = {
waitingTxnID?: string;
Expand Down
4 changes: 2 additions & 2 deletions pkg/ui/workspaces/cluster-ui/src/api/databaseDetailsApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,13 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import moment from "moment-timezone";
import { cockroach } from "@cockroachlabs/crdb-protobuf-client";
import moment from "moment-timezone";

import { IndexUsageStatistic, recommendDropUnusedIndex } from "../insights";
import { getLogger, indexUnusedDuration, maybeError } from "../util";

import { Format, Identifier, QualifiedIdentifier } from "./safesql";
import {
combineQueryErrors,
createSqlExecutionRequest,
Expand All @@ -30,7 +31,6 @@ import {
SqlTxnResult,
txnResultIsEmpty,
} from "./sqlApi";
import { Format, Identifier, QualifiedIdentifier } from "./safesql";
import { fromHexString, withTimeout } from "./util";

const { ZoneConfig } = cockroach.config.zonepb;
Expand Down
4 changes: 2 additions & 2 deletions pkg/ui/workspaces/cluster-ui/src/api/indexDetailsApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ import {
StatementRawFormat,
} from "src/api";

import { TimeScale, toRoundedDateRange } from "../timeScaleDropdown";
import { AggregateStatistics } from "../statementsTable";
import { INTERNAL_APP_NAME_PREFIX } from "../activeExecutions/activeStatementUtils";
import { AggregateStatistics } from "../statementsTable";
import { TimeScale, toRoundedDateRange } from "../timeScaleDropdown";

export type TableIndexStatsRequest =
cockroach.server.serverpb.TableIndexStatsRequest;
Expand Down
2 changes: 1 addition & 1 deletion pkg/ui/workspaces/cluster-ui/src/api/schemaInsightsApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import {
} from "../insights";
import { HexStringToInt64String, indexUnusedDuration } from "../util";

import { QuoteIdentifier } from "./safesql";
import {
SqlExecutionRequest,
SqlTxnResult,
Expand All @@ -25,7 +26,6 @@ import {
SqlApiResponse,
formatApiResult,
} from "./sqlApi";
import { QuoteIdentifier } from "./safesql";

// Export for db-console import from clusterUiApi.
export type { InsightRecommendation } from "../insights";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@

import moment from "moment-timezone";

import { Duration } from "src/util/format";
import {
createSqlExecutionRequest,
executeInternalSql,
Expand All @@ -19,6 +18,7 @@ import {
SqlTxnResult,
txnResultSetIsEmpty,
} from "src/api";
import { Duration } from "src/util/format";

export type StatementDiagnosticsReport = {
id: string;
Expand Down
2 changes: 1 addition & 1 deletion pkg/ui/workspaces/cluster-ui/src/api/stmtInsightsApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ import {
} from "src/insights/types";
import { INTERNAL_APP_NAME_PREFIX } from "src/util/constants";

import { FixFingerprintHexValue } from "../util";
import { getInsightsFromProblemsAndCauses } from "../insights/utils";
import { FixFingerprintHexValue } from "../util";

import { getContentionDetailsApi } from "./contentionApi";
import {
Expand Down
4 changes: 2 additions & 2 deletions pkg/ui/workspaces/cluster-ui/src/api/tableDetailsApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,13 @@
// by the Apache License, Version 2.0, included in the file
// licenses/APL.txt.

import moment from "moment-timezone";
import { cockroach } from "@cockroachlabs/crdb-protobuf-client";
import moment from "moment-timezone";

import { IndexUsageStatistic, recommendDropUnusedIndex } from "../insights";
import { getLogger, indexUnusedDuration, maybeError } from "../util";

import { Format, Identifier, Join, SQL } from "./safesql";
import { fromHexString, withTimeout } from "./util";
import {
combineQueryErrors,
executeInternalSql,
Expand All @@ -30,6 +29,7 @@ import {
SqlTxnResult,
txnResultIsEmpty,
} from "./sqlApi";
import { fromHexString, withTimeout } from "./util";

const { ZoneConfig } = cockroach.config.zonepb;
const { ZoneConfigurationLevel } = cockroach.server.serverpb;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import {
} from "../insights";
import { maybeError } from "../util";

import { getTxnInsightsContentionDetailsApi } from "./contentionApi";
import {
executeInternalSql,
isMaxSizeError,
Expand All @@ -34,7 +35,6 @@ import {
TxnInsightsResponseRow,
} from "./txnInsightsApi";
import { makeInsightsSqlRequest } from "./txnInsightsUtils";
import { getTxnInsightsContentionDetailsApi } from "./contentionApi";

export type TxnInsightDetailsRequest = {
txnExecutionID: string;
Expand Down
Loading

0 comments on commit c5f7261

Please sign in to comment.