Skip to content

Commit

Permalink
fix(clickhouse-driver): merging config with custom queryOptions which…
Browse files Browse the repository at this point in the history
… were not passing along the database (#100)
  • Loading branch information
cameronbraid authored and paveltiunov committed May 2, 2019
1 parent ff1b65d commit dedc279
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions packages/cubejs-clickhouse-driver/driver/ClickHouseDriver.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,14 @@ class ClickHouseDriver extends BaseDriver {
...config
};
this.pool = genericPool.createPool({
create: () => new ClickHouse(Object.assign({}, this.config, {
create: () => new ClickHouse({
...this.config,
queryOptions: {
join_use_nulls: 1,
session_id: uuid()
session_id: uuid(),
...this.config.queryOptions,
}
})),
}),
destroy: () => Promise.resolve(),
validate: async (connection) => {
try {
Expand Down

0 comments on commit dedc279

Please sign in to comment.