admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix clickhouse migration (#11699)

Browse Source 
It looks like Clickhouse Cloud uses a different config var than the docs
This commit is contained in:
Félix Malfait
2025-04-23 15:09:23 +02:00
committed by GitHub
parent 457c6ed113
commit e1054b0474
1 changed files with 25 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
packages/twenty-server/src/database/clickhouse/migrations/run-migrations.ts
View File

@ -2,7 +2,7 @@
import fs from 'fs'; import fs from 'fs';
import path from 'path'; import path from 'path';
import { createClient, ClickHouseClient } from '@clickhouse/client'; import { ClickHouseClient, createClient } from '@clickhouse/client';
import { config } from 'dotenv'; import { config } from 'dotenv';
config({ config({
@ -29,9 +29,30 @@ async function ensureDatabaseExists() {
await client.command({ await client.command({
query: `CREATE DATABASE IF NOT EXISTS "${database}"`, query: `CREATE DATABASE IF NOT EXISTS "${database}"`,
}); });
await client.command({
query: `SET enable_json_type = 1`, let jsonTypeEnabled = false;
});
try {
await client.command({
query: `SET enable_json_type = 1`,
});
jsonTypeEnabled = true;
} catch (error) {
// Intentionally empty - will try alternative method
}
try {
await client.command({
query: `SET allow_experimental_json_type = 1`,
});
jsonTypeEnabled = true;
} catch (error) {
// Intentionally empty - failure handled by jsonTypeEnabled check
}
if (!jsonTypeEnabled) {
console.error('Failed to enable JSON type');
}
await client.close(); await client.close();
} }