akarazhev opened a new pull request, #75425:
URL: https://github.com/apache/airflow/pull/75425

   `SnowflakeHook.run(autocommit=False)` loses its transaction when 
`session_parameters` is set, even without `AUTOCOMMIT` in it. The connector 
merges the server's session parameters into the dict passed to `connect()`, so 
the hook then finds `AUTOCOMMIT` there and skips `ALTER SESSION SET 
autocommit=False` and the final `COMMIT`. The hook now passes the connector a 
copy.
   
   The new test fails on main and passes with the fix; the Snowflake provider 
unit tests pass.
   
   closes: #75419
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code (Opus 5.5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to