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

chore: bump OL provider dependencies versions #42059

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

kacpermuda
Copy link
Contributor

Just a regular bump of the OpenLineage libraries to take advantage of latest features.


^ 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.

@potiuk
Copy link
Member

potiuk commented Sep 6, 2024

I am afraid latest openlineage release broke main - see here https://github.com/apache/airflow/actions/runs/10734311195/job/29769445166

@potiuk
Copy link
Member

potiuk commented Sep 6, 2024

cc: @kacpermuda @mobuchowski

@topherinternational
Copy link
Contributor

I am afraid latest openlineage release broke main - see here https://github.com/apache/airflow/actions/runs/10734311195/job/29769445166

I saw the same failures in a branch I'm working on, I think #42054's change to the Dataset extra field is the problem? A bunch of tests that expect the default val to be None are failing bc the default is now {}.

@kacpermuda
Copy link
Contributor Author

Yes, i don't think the OL is to blame here 😄

@potiuk
Copy link
Member

potiuk commented Sep 6, 2024

Yes, i don't think the OL is to blame here 😄

Ups

@kacpermuda kacpermuda force-pushed the chore-bump-ol-provider-dependencies branch from 4e81c18 to e371ac0 Compare September 6, 2024 13:33
@mobuchowski mobuchowski added the full tests needed We need to run full set of tests for this PR to merge label Sep 6, 2024
@potiuk potiuk merged commit e05c035 into apache:main Sep 10, 2024
109 checks passed
@kacpermuda kacpermuda deleted the chore-bump-ol-provider-dependencies branch September 12, 2024 17:28
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers full tests needed We need to run full set of tests for this PR to merge provider:openlineage AIP-53
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants