Skip to content

Commit 5b1fd59

Browse files
committed
fix(clickhouse-driver): Remove SQL from ClickHouse error messages
1 parent a3ed6ea commit 5b1fd59

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

packages/cubejs-clickhouse-driver/src/ClickHouseDriver.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ export class ClickHouseDriver extends BaseDriver implements DriverInterface {
260260
return results;
261261
} catch (e) {
262262
// TODO replace string formatting with proper cause
263-
throw new Error(`Query failed; cause: ${e}; query id: ${queryId}; SQL: ${query}`);
263+
throw new Error(`Query failed: ${e}; query id: ${queryId}`);
264264
}
265265
});
266266
}
@@ -409,7 +409,7 @@ export class ClickHouseDriver extends BaseDriver implements DriverInterface {
409409
} catch (e) {
410410
await client.close();
411411
// TODO replace string formatting with proper cause
412-
throw new Error(`Stream query failed; cause: ${e}; query id: ${queryId}; SQL: ${query}`);
412+
throw new Error(`Stream query failed: ${e}; query id: ${queryId}`);
413413
}
414414
}
415415

@@ -554,7 +554,7 @@ export class ClickHouseDriver extends BaseDriver implements DriverInterface {
554554
await this.command(createTableSql);
555555
} catch (e) {
556556
// TODO replace string formatting with proper cause
557-
throw new Error(`Create table failed; cause: ${e}; SQL: ${createTableSql}`);
557+
throw new Error(`Create table failed: ${e}`);
558558
}
559559
}
560560

0 commit comments

Comments
 (0)