Skip to content

Commit 29bc2fc

Browse files
Updating snowflake driver to respect case by default with added override.
1 parent d0bcf15 commit 29bc2fc

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

packages/cubejs-snowflake-driver/src/SnowflakeDriver.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -453,7 +453,10 @@ export class SnowflakeDriver extends BaseDriver implements DriverInterface {
453453

454454
await this.execute(connection, 'ALTER SESSION SET TIMEZONE = \'UTC\'', [], false);
455455
await this.execute(connection, `ALTER SESSION SET STATEMENT_TIMEOUT_IN_SECONDS = ${this.config.executionTimeout}`, [], false);
456-
if (this.ignoreCase) {
456+
457+
// We only want to ignore the case if someone sets the value to false explicitly since the default assumption
458+
// is that casing matters
459+
if (!this.ignoreCase) {
457460
await this.execute(connection, 'ALTER SESSION SET QUOTED_IDENTIFIERS_IGNORE_CASE = FALSE', [], false);
458461
}
459462
return connection;

0 commit comments

Comments
 (0)