Skip to content

remove snowflake query timeout - #36452

Merged
Joe Bell (jbfbell) merged 1 commit into
masterfrom
jbfbell/remove_snowflake_query_timeout
Mar 25, 2024
Merged

remove snowflake query timeout#36452
Joe Bell (jbfbell) merged 1 commit into
masterfrom
jbfbell/remove_snowflake_query_timeout

Conversation

@jbfbell

@jbfbell Joe Bell (jbfbell) commented Mar 25, 2024

Copy link
Copy Markdown
Contributor

We have provided a snowflake query timeout of 3 hours which users are running into for large or long running syncs. Removing this query timeout so queries will wait indefinitely.

@vercel

vercel Bot commented Mar 25, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs 🛑 Canceled (Inspect) Mar 25, 2024 4:55pm

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join Joe Bell (@jbfbell) and the rest of your teammates on Graphite Graphite

@edgao Edward Gao (edgao) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

happy to ship as-is, wondering if we should expose this as a config option eventually

@jbfbell
Joe Bell (jbfbell) merged commit 447c967 into master Mar 25, 2024
@jbfbell
Joe Bell (jbfbell) deleted the jbfbell/remove_snowflake_query_timeout branch March 25, 2024 17:37
@jbfbell

Copy link
Copy Markdown
Contributor Author

🔥

happy to ship as-is, wondering if we should expose this as a config option eventually

Edward Gao (@edgao) agreed - supposedly a user could configure this via the JDBC url params but its probably worth verifying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/destination/snowflake

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants