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

[Internal] Update Jobs GetRun API to support paginated responses for jobs and ForEach tasks #1089

Merged
merged 2 commits into from
Nov 8, 2024

Conversation

gkiko10
Copy link
Contributor

@gkiko10 gkiko10 commented Nov 8, 2024

What changes are proposed in this pull request?

Introduces extension for jobs GetRun call that paginates tasks and iterations arrays in the response. This change is necessary to prepare for jobs API 2.2 release that serves paginated response. Pagination is over once the next_page_token is absent from the response. The pagination logic is not exposed to the customer.

How is this tested?

Unit test and manual test. I wrote go code that called the new logic and verified that the returned result contained full object when the API was paginated.

Copy link

github-actions bot commented Nov 8, 2024

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: 1089
  • Commit SHA: d5bbaa08b738d9ad7415e15ae8dd4f70fdf2cab6

Checks will be approved automatically on success.

@eng-dev-ecosystem-bot
Copy link
Collaborator

Test Details: go/deco-tests/11744067074

@renaudhartert-db renaudhartert-db added this pull request to the merge queue Nov 8, 2024
Merged via the queue into databricks:main with commit 1981951 Nov 8, 2024
11 checks passed
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.

3 participants