Reset the session before logging

Snowflake-Labs / schemachange

A Database Change Management tool for Snowflake

Apache License 2.0

481 stars 219 forks source link

Reset the session before logging #162

Closed nwea-yeti-jw closed 10 months ago

nwea-yeti-jw commented 1 year ago

Issue #161 The session needs to be reset in case role, database, or warehouse were set within the migration script.

pavlovecjakePFG commented 1 year ago

This PR fixes a bug that was introduced in one of the recent versions. I recently attempted to upgrade to the newest version (current = 3.4.1) and had to roll back because of this bug. If a role is assumed in a script that does not have access to modify the change history table, then you get an error.

temyers commented 1 year ago

Any update on this?

This caused a breaking change preventing upgrade from a previous version.

MaxKrog commented 10 months ago

Bump! We're having the same issue and a merge of this fix would be highly appreciated.

Thanks for a great tool.

sfc-gh-jhansen commented 10 months ago

These changes were deployed in v3.6.0, PR #195. Thanks everyone.