Skip to content

Commit

Permalink
remove snowflake query timeout (#36452)
Browse files Browse the repository at this point in the history
  • Loading branch information
jbfbell authored Mar 25, 2024
1 parent 4b1ebb7 commit 447c967
Show file tree
Hide file tree
Showing 3 changed files with 180 additions and 181 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ data:
connectorSubtype: database
connectorType: destination
definitionId: 424892c4-daac-4491-b35d-c6688ba547ba
dockerImageTag: 3.6.2
dockerImageTag: 3.6.3
dockerRepository: airbyte/destination-snowflake
documentationUrl: https://docs.airbyte.com/integrations/destinations/snowflake
githubIssueLabel: destination-snowflake
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ public class SnowflakeDatabase {
private static final int PAUSE_BETWEEN_TOKEN_REFRESH_MIN = 7; // snowflake access token TTL is 10min and can't be modified

private static final Duration NETWORK_TIMEOUT = Duration.ofMinutes(1);
private static final Duration QUERY_TIMEOUT = Duration.ofHours(3);
private static final SnowflakeSQLNameTransformer nameTransformer = new SnowflakeSQLNameTransformer();
private static final String DRIVER_CLASS_NAME = "net.snowflake.client.jdbc.SnowflakeDriver";

Expand Down Expand Up @@ -123,7 +122,6 @@ public static HikariDataSource createDataSource(final JsonNode config, final Str
properties.put(JdbcUtils.SCHEMA_KEY, nameTransformer.getIdentifier(config.get(JdbcUtils.SCHEMA_KEY).asText()));

properties.put("networkTimeout", Math.toIntExact(NETWORK_TIMEOUT.toSeconds()));
properties.put("queryTimeout", Math.toIntExact(QUERY_TIMEOUT.toSeconds()));
// allows queries to contain any number of statements.
properties.put("MULTI_STATEMENT_COUNT", 0);

Expand Down
Loading

0 comments on commit 447c967

Please sign in to comment.