Skip to content

fix: parse WaitTimeout and use it as poll timeout in ExecuteAndWait#1730

Open
gourabsingha1 wants to merge 3 commits into
databricks:mainfrom
gourabsingha1:fix/execute-and-wait-timeout
Open

fix: parse WaitTimeout and use it as poll timeout in ExecuteAndWait#1730
gourabsingha1 wants to merge 3 commits into
databricks:mainfrom
gourabsingha1:fix/execute-and-wait-timeout

Conversation

@gourabsingha1

Copy link
Copy Markdown

Closes #1448. Parses request.WaitTimeout duration and uses it for the retries.Poll loop inside ExecuteAndWait, falling back to 20m if unspecified or invalid.

@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1730
  • Commit SHA: f9d4ede4b618821bbabc5f4fe10ea06d3eff9b81

Checks will be approved automatically on success.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ISSUE] ExecuteAndWait uses a hard-coded 20 minute timeout

1 participant