Skip to content

Commit d6f8058

Browse files
authored
chore: bump @clickhouse/client to v1.12.1 + centralize @clickhouse/client (#1110)
1. deprecate unused `packages/api/src/clickhouse` 2. deprecate unused route `/datasources` 3. introduce `getJSNativeCreateClient` in common-utils 4. uninstall `@clickhouse/client` in api package 5. uninstall `@clickhouse/client` + `@clickhouse/client-web` in app package 6. bump `@clickhouse/client` in common-utils package to v1.12.1
1 parent f800fd1 commit d6f8058

25 files changed

+146
-691
lines changed

.changeset/empty-maps-kiss.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
---
2+
"@hyperdx/common-utils": patch
3+
"@hyperdx/api": patch
4+
"@hyperdx/app": patch
5+
---
6+
7+
- deprecate unused packages/api/src/clickhouse
8+
- deprecate unused route /datasources
9+
- introduce getJSNativeCreateClient in common-utils
10+
- uninstall @clickhouse/client in api package
11+
- uninstall @clickhouse/client + @clickhouse/client-web in app package
12+
- bump @clickhouse/client in common-utils package to v1.12.1

packages/api/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
"node": ">=22.16.0"
88
},
99
"dependencies": {
10-
"@clickhouse/client": "^0.2.10",
1110
"@esm2cjs/p-queue": "^7.3.0",
1211
"@hyperdx/common-utils": "^0.3.2",
1312
"@hyperdx/lucene": "^3.1.1",

packages/api/src/api-app.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@ app.use('/dashboards', isUserAuthenticated, routers.dashboardRouter);
8888
app.use('/me', isUserAuthenticated, routers.meRouter);
8989
app.use('/team', isUserAuthenticated, routers.teamRouter);
9090
app.use('/webhooks', isUserAuthenticated, routers.webhooksRouter);
91-
app.use('/datasources', isUserAuthenticated, routers.datasourceRouter);
9291
app.use('/connections', isUserAuthenticated, connectionsRouter);
9392
app.use('/sources', isUserAuthenticated, sourcesRouter);
9493
app.use('/saved-search', isUserAuthenticated, savedSearchRouter);

packages/api/src/clickhouse/__tests__/renderChartConfig.test.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,14 @@
33
import { ChSql, ClickhouseClient } from '@hyperdx/common-utils/dist/clickhouse';
44
import { getMetadata } from '@hyperdx/common-utils/dist/metadata';
55
import { renderChartConfig } from '@hyperdx/common-utils/dist/renderChartConfig';
6-
import _ from 'lodash';
7-
import ms from 'ms';
8-
96
import {
107
AggregateFunctionSchema,
118
DerivedColumn,
129
MetricsDataType,
13-
} from '@/../../common-utils/dist/types';
10+
} from '@hyperdx/common-utils/dist/types';
11+
import _ from 'lodash';
12+
import ms from 'ms';
13+
1414
import * as config from '@/config';
1515
import { createTeam } from '@/controllers/team';
1616
import {
@@ -26,7 +26,6 @@ import {
2626
} from '@/fixtures';
2727
import Connection from '@/models/connection';
2828
import { Source } from '@/models/source';
29-
import { AggregationTemporality } from '@/utils/logParser';
3029

3130
const TEST_METRIC_TABLES = {
3231
sum: DEFAULT_METRICS_TABLE.SUM,

0 commit comments

Comments
 (0)