Skip to content

[pull] main from snowflakedb:main#165

Merged
pull[bot] merged 1 commit intoitsbrex:mainfrom
snowflakedb:main
Apr 3, 2026
Merged

[pull] main from snowflakedb:main#165
pull[bot] merged 1 commit intoitsbrex:mainfrom
snowflakedb:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 3, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

fix: scope client_session_keep_alive to token auth only (SNOW-3322928)

PR #2832 fixed the parameter name from `using_session_keep_alive` to
`client_session_keep_alive`, which inadvertently enabled heartbeat
threads for all connections. In CI with pytest-xdist (in-process
invocations), connections accumulate across tests, sessions pile up
on the shared Snowflake account, and the backend throttles — causing
integration tests to go from ~40min to 6+ hours and timing out.

Scope `client_session_keep_alive` to only apply when authenticating
with session/master tokens (the original intent). Also add
timeout-minutes: 60 to integration test workflow jobs as a safety net.
@pull pull bot locked and limited conversation to collaborators Apr 3, 2026
@pull pull bot added the ⤵️ pull label Apr 3, 2026
@pull pull bot merged commit 0fd2247 into itsbrex:main Apr 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant