Skip to content

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

Signed-off-by: Kacper Muda <mudakacper@gmail.com>
@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
@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
Signed-off-by: Kacper Muda <mudakacper@gmail.com>
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