Set default isolation level to AUTOCOMMIT for snowflake

[sfc-gh-dhuang]

Description

This is needed after snowflake-sqlalchemy >= 1.7.2 because of snowflakedb/snowflake-sqlalchemy#555

Other details good to know for developers

Please include any other details of this change useful for TruLens developers.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to
  not work as expected)
• New Tests
• This change includes re-generated golden test results
• This change requires a documentation update

---

Important

Set default isolation level to AUTOCOMMIT for Snowflake connections in SQLAlchemyDB to ensure compatibility with snowflake-sqlalchemy >= 1.7.2.

• Behavior:
  • Set default isolation_level to AUTOCOMMIT for Snowflake connections in _reload_engine() of SQLAlchemyDB.
  • Applies only if snowflake is detected in engine_params['url'].
• Compatibility:
  • Ensures compatibility with snowflake-sqlalchemy version 1.7.2 and above.

^{This description was created by for 41cea5b. It will automatically update as commits are pushed.}

[sfc-gh-jreini]

curious how this works. is the isolation level somehow retained after you dispose of the temporary engine?

[sfc-gh-dhuang]

yes, isolation_level will be set in self.engine_params and this will be used (in the case of snowflake being the DB) when creating the new engine on lines below

For context, I agree this is a bit ugly but I think the overhead of creating and disposing a temp_engine is okay as it's just opening and closing a DB connection.