Skip to content

Commit

Permalink
SNOW-1437655: Set auth timeout to 0 in not auth endpoint calls
Browse files Browse the repository at this point in the history
  • Loading branch information
sfc-gh-dprzybysz committed May 29, 2024
1 parent d84639c commit 9168cdf
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions src/main/java/net/snowflake/client/core/SFSession.java
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ private JsonNode getQueryMetadata(String queryID) throws SQLException {
HttpUtil.executeGeneralRequest(
get,
loginTimeout,
authTimeout,
0,
(int) httpClientSocketTimeout.toMillis(),
maxHttpRetries,
getHttpClientKey());
Expand Down Expand Up @@ -967,7 +967,7 @@ protected void heartbeat() throws SFException, SQLException {
HttpUtil.executeGeneralRequest(
postRequest,
SF_HEARTBEAT_TIMEOUT,
authTimeout,
0,
(int) httpClientSocketTimeout.toMillis(),
0,
getHttpClientKey());
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/net/snowflake/client/core/SessionUtil.java
Original file line number Diff line number Diff line change
Expand Up @@ -1082,7 +1082,7 @@ static void closeSession(SFLoginInput loginInput) throws SFException, SnowflakeS
HttpUtil.executeGeneralRequest(
postRequest,
loginInput.getLoginTimeout(),
loginInput.getAuthTimeout(),
0,
loginInput.getSocketTimeoutInMillis(),
0,
loginInput.getHttpClientSettingsKey());
Expand Down
4 changes: 2 additions & 2 deletions src/main/java/net/snowflake/client/core/StmtUtil.java
Original file line number Diff line number Diff line change
Expand Up @@ -393,8 +393,8 @@ public static StmtOutput execute(StmtInput stmtInput, ExecTimeTelemetryData exec
HttpUtil.executeRequest(
httpRequest,
stmtInput.networkTimeoutInMillis / 1000,
stmtInput.socketTimeout,
0,
stmtInput.socketTimeout,
stmtInput.maxRetries,
stmtInput.injectSocketTimeout,
stmtInput.canceling,
Expand Down Expand Up @@ -636,8 +636,8 @@ protected static String getQueryResult(String getResultPath, StmtInput stmtInput
return HttpUtil.executeRequest(
httpRequest,
stmtInput.networkTimeoutInMillis / 1000,
stmtInput.socketTimeout,
0,
stmtInput.socketTimeout,
stmtInput.maxRetries,
0,
stmtInput.canceling,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -365,7 +365,7 @@ private boolean sendBatch() throws IOException {
: HttpUtil.executeGeneralRequest(
post,
TELEMETRY_HTTP_RETRY_TIMEOUT_IN_SEC,
this.session.getAuthTimeout(),
0,
this.session.getHttpClientSocketTimeout(),
0,
this.session.getHttpClientKey());
Expand Down

0 comments on commit 9168cdf

Please sign in to comment.