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

feat: Add OpenLineage support for non-query jobs in BigQueryInsertJobOperator #44996

Merged

Conversation

kacpermuda
Copy link
Contributor

@kacpermuda kacpermuda commented Dec 17, 2024

This PR adds OpenLineage support for all BQ job types other than QUERY in BigQueryInsertJobOperator (that was already supported).


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Dec 17, 2024
@kacpermuda kacpermuda force-pushed the feat-ol-bq-insert-job-other-job-types branch 4 times, most recently from 9deeff7 to 344f1ce Compare December 19, 2024 09:47
@kacpermuda kacpermuda force-pushed the feat-ol-bq-insert-job-other-job-types branch from 344f1ce to 1937e43 Compare December 30, 2024 15:37
@kacpermuda kacpermuda marked this pull request as ready for review December 30, 2024 16:10
@eladkal eladkal requested a review from mobuchowski January 1, 2025 07:16
@kacpermuda kacpermuda force-pushed the feat-ol-bq-insert-job-other-job-types branch 2 times, most recently from d7301d1 to 3fb0eb7 Compare January 2, 2025 10:50
@potiuk
Copy link
Member

potiuk commented Jan 2, 2025

@kacpermuda - can you please rebase that one -> we found and issue with @jscheffl with the new caching scheme - fixed in #45347 that would run "main" version of the tests. I am asking in all affected PRs to rebase.

@potiuk potiuk force-pushed the feat-ol-bq-insert-job-other-job-types branch from 3fb0eb7 to 286fd8c Compare January 2, 2025 12:22
@potiuk
Copy link
Member

potiuk commented Jan 2, 2025

Actually.. I rebased it myself :)

@kacpermuda kacpermuda force-pushed the feat-ol-bq-insert-job-other-job-types branch from 286fd8c to 5a0f236 Compare January 2, 2025 14:10
@kacpermuda kacpermuda force-pushed the feat-ol-bq-insert-job-other-job-types branch from 5a0f236 to 2891af3 Compare January 3, 2025 07:57
@mobuchowski mobuchowski merged commit 6f9cf5a into apache:main Jan 3, 2025
60 checks passed
@kacpermuda kacpermuda deleted the feat-ol-bq-insert-job-other-job-types branch January 3, 2025 10:08
LefterisXefteris pushed a commit to LefterisXefteris/airflow that referenced this pull request Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants