Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Databricks Add new hook method#62822

Merged
potiuk merged 1 commit into
apache:mainfrom
Zinkue:feature/databricks/jobs-api-pagination
Mar 12, 2026
Merged

Databricks Add new hook method#62822
potiuk merged 1 commit into
apache:mainfrom
Zinkue:feature/databricks/jobs-api-pagination

Conversation

@Zinkue
Copy link
Copy Markdown
Contributor

@Zinkue Zinkue commented Mar 3, 2026

Description

As part of the migration of the Databricks API from 2.1 to 2.2, the get_run method of the hook did not support pagination for specific array properties.
We can create Databricks Workflows with more than 100 tasks, but the monitoring of tasks in Airflow fails in random tasks because the get_run retrieves only 100 tasks. Therefore when we try to check the status of a task we get an KeyError

Solution

Added a new hook method get_run_tasks to support pagination on the tasks array property and solve the problem mentioned above.

Links

Databricks API Get a single job run

related: #59189
related: #59217


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@Zinkue
Copy link
Copy Markdown
Contributor Author

Zinkue commented Mar 3, 2026

@jroachgolf84 If you're free and interested, since you did the initial API migration, could you take a quick look? Many thanks in advance!

@Zinkue Zinkue force-pushed the feature/databricks/jobs-api-pagination branch 2 times, most recently from 0cc5edd to 1f48d6b Compare March 7, 2026 17:57
@Zinkue Zinkue force-pushed the feature/databricks/jobs-api-pagination branch from 1f48d6b to 417170d Compare March 10, 2026 21:24
@jroachgolf84
Copy link
Copy Markdown
Collaborator

Yep, taking a quick look now!

Copy link
Copy Markdown
Collaborator

@jroachgolf84 jroachgolf84 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM! I'm not a committer, so my approval doesn't hold weight in terms of approving the PR.

@potiuk potiuk merged commit 861d6bc into apache:main Mar 12, 2026
92 checks passed
PascalEgn pushed a commit to PascalEgn/airflow that referenced this pull request Mar 12, 2026
Pyasma pushed a commit to Pyasma/airflow that referenced this pull request Mar 13, 2026
@Zinkue Zinkue deleted the feature/databricks/jobs-api-pagination branch May 5, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants