Updating snowflake driver to respect case by default with added override.

mike-taylor-nomihealth · mike-taylor-nomihealth · commit 9b7294941d39 · 2025-01-07T14:29:12.000-07:00
diff --git a/packages/cubejs-snowflake-driver/src/SnowflakeDriver.ts b/packages/cubejs-snowflake-driver/src/SnowflakeDriver.ts
@@ -453,7 +453,7 @@ export class SnowflakeDriver extends BaseDriver implements DriverInterface {
 
       await this.execute(connection, 'ALTER SESSION SET TIMEZONE = \'UTC\'', [], false);
       await this.execute(connection, `ALTER SESSION SET STATEMENT_TIMEOUT_IN_SECONDS = ${this.config.executionTimeout}`, [], false);
-      if (this.ignoreCase) {
+      if (!this.ignoreCase) {
         await this.execute(connection, 'ALTER SESSION SET QUOTED_IDENTIFIERS_IGNORE_CASE = FALSE', [], false);
       }
       return connection;

Original file line number	Diff line number	Diff line change
`@@ -453,7 +453,7 @@ export class SnowflakeDriver extends BaseDriver implements DriverInterface {`
`453`	`453`
`454`	`454`	`await this.execute(connection, 'ALTER SESSION SET TIMEZONE = \'UTC\'', [], false);`
`455`	`455`	await this.execute(connection, `ALTER SESSION SET STATEMENT_TIMEOUT_IN_SECONDS = ${this.config.executionTimeout}`, [], false);
`456`		`- if (this.ignoreCase) {`
	`456`	`+ if (!this.ignoreCase) {`
`457`	`457`	`await this.execute(connection, 'ALTER SESSION SET QUOTED_IDENTIFIERS_IGNORE_CASE = FALSE', [], false);`
`458`	`458`	`}`
`459`	`459`	`return connection;`