Skip to content

Fix failing OpenLineage emition for InsertBigQueryOperator - #44650

Merged
mobuchowski merged 6 commits into
apache:mainfrom
spapi17:bqop-ol
Dec 10, 2024
Merged

Fix failing OpenLineage emition for InsertBigQueryOperator #44650
mobuchowski merged 6 commits into
apache:mainfrom
spapi17:bqop-ol

Conversation

@spapi17

@spapi17 spapi17 commented Dec 4, 2024

Copy link
Copy Markdown
Contributor

InsertBigQueryOperator OpenLineage emition fails with google.api_core.exceptions.NotFound when a non-default location is used, added location argument in BigQuery client to prevent this.


^ 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 4, 2024
@spapi17
spapi17 marked this pull request as ready for review December 4, 2024 16:06
@spapi17
spapi17 requested a review from moiseenkov December 9, 2024 13:05
@amoghrajesh

Copy link
Copy Markdown
Contributor

cc @mobuchowski @kacpermuda
I am approving the workflow

@mobuchowski
mobuchowski merged commit 5079f7d into apache:main Dec 10, 2024
@boring-cyborg

boring-cyborg Bot commented Dec 10, 2024

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
)

* specify location in BigQuery client

* add location in test operator class
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.

6 participants