Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SNOW-878067: Retry Strategy #803

Merged
merged 24 commits into from
Nov 6, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
68e4dc1
SNOW-878067: Add retry strategy for supported endpoints
sfc-gh-ext-simba-lf Oct 26, 2023
b7dc6bd
SNOW-878067: Add tests for retry strategy
sfc-gh-ext-simba-lf Oct 26, 2023
59f1f7e
Merge branch 'master' of https://github.com/snowflakedb/snowflake-con…
sfc-gh-ext-simba-lf Oct 26, 2023
5d7f9e1
SNOW-878067: Fix variable name
sfc-gh-ext-simba-lf Oct 26, 2023
77611e6
SNOW-878067: Increase delta for async timeout test
sfc-gh-ext-simba-lf Oct 26, 2023
9639f43
SNOW-878067: Increase delta for async timeout test
sfc-gh-ext-simba-lf Oct 26, 2023
a441122
SNOW-878067: Increase delta for timeout test
sfc-gh-ext-simba-lf Oct 26, 2023
ddd961e
SNOW-878067: Refactor variable for connection timeout
sfc-gh-ext-simba-lf Oct 27, 2023
a203e19
SNOW-878067: Refactor variable for connection timeout
sfc-gh-ext-simba-lf Oct 27, 2023
949c257
Merge branch 'master' of https://github.com/snowflakedb/snowflake-con…
sfc-gh-ext-simba-lf Oct 27, 2023
da0d1c6
Update CONNECTION_TIMEOUT default value from 120 to 300
sfc-gh-ext-simba-lf Oct 31, 2023
8007e5e
SNOW-878067: Update retry logic
sfc-gh-ext-simba-lf Nov 3, 2023
5b926b7
Update README.md with new parameter "RETRY_TIMEOUT"
sfc-gh-ext-simba-lf Nov 3, 2023
2866dec
SNOW-878067: Modify timeout test cases
sfc-gh-ext-simba-lf Nov 3, 2023
692bb36
Merge branch 'SNOW-878067-Retry-Strategy' of https://github.com/snowf…
sfc-gh-ext-simba-lf Nov 3, 2023
d0a445e
SNOW-878067: Modify timeout test cases
sfc-gh-ext-simba-lf Nov 3, 2023
ce6b2b6
SNOW-878067: Modify timeout test cases
sfc-gh-ext-simba-lf Nov 3, 2023
98f0abe
SNOW-878067: Modify timeout test cases
sfc-gh-ext-simba-lf Nov 3, 2023
b25afec
SNOW-878067: Update comment
sfc-gh-ext-simba-lf Nov 3, 2023
13fa45f
SNOW-878067: Modify timeout test cases
sfc-gh-ext-simba-lf Nov 3, 2023
5e9931e
SNOW-878067: Refactor retry logic
sfc-gh-ext-simba-lf Nov 3, 2023
4cc94f7
SNOW-878067: Set maxHttpRetries to 0 to reach the defined timeout
sfc-gh-ext-simba-lf Nov 3, 2023
ef12cef
SNOW-878067: Remove OS Platform restriction on timeout tests
sfc-gh-ext-simba-lf Nov 3, 2023
4f511f4
SNOW-878067: Refactor setting which timeout to use
sfc-gh-ext-simba-lf Nov 4, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,8 @@ The following table lists all valid connection properties:
| SCHEMA | No | |
| USER | Depends | If AUTHENTICATOR is set to `externalbrowser` this is optional. For native SSO through Okta, set this to the login name for your identity provider (IdP). |
| WAREHOUSE | No | |
| CONNECTION_TIMEOUT | No | Total timeout in seconds when connecting to Snowflake. The default is 120 seconds |
| CONNECTION_TIMEOUT | No | Total timeout in seconds when connecting to Snowflake. The default is 300 seconds |
| RETRY_TIMEOUT | No | Total timeout in seconds for supported endpoints of retry policy. The default is 300 seconds. The value can only be increased from the default value or set to 0 for infinite timeout |
| MAXHTTPRETRIES | No | Maximum number of times to retry failed HTTP requests (default: 7). You can set `MAXHTTPRETRIES=0` to remove the retry limit, but doing so runs the risk of the .NET driver infinitely retrying failed HTTP calls. |
| CLIENT_SESSION_KEEP_ALIVE | No | Whether to keep the current session active after a period of inactivity, or to force the user to login again. If the value is `true`, Snowflake keeps the session active indefinitely, even if there is no activity from the user. If the value is `false`, the user must log in again after four hours of inactivity. The default is `false`. Setting this value overrides the server session property for the current session. |
| DISABLERETRY | No | Set this property to `true` to prevent the driver from reconnecting automatically when the connection fails or drops. The default value is `false`. |
Expand Down
Loading
Loading