-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
openlineage: migrate OpenLineage provider to V2 facets. #39530
base: main
Are you sure you want to change the base?
openlineage: migrate OpenLineage provider to V2 facets. #39530
Conversation
2e86fb5
to
9947104
Compare
9947104
to
fc72488
Compare
0479252
to
e43e266
Compare
I ran |
e43e266
to
74bc580
Compare
#39747 should fix the docker test in "Latest Botocore". |
Signed-off-by: Jakub Dardzinski <kuba0221@gmail.com>
Signed-off-by: Jakub Dardzinski <kuba0221@gmail.com>
Signed-off-by: Jakub Dardzinski <kuba0221@gmail.com>
Signed-off-by: Jakub Dardzinski <kuba0221@gmail.com>
74bc580
to
feaa46c
Compare
@JDarDagran we probably need to bump cross-provider version dependencies. If someone has older OL provider version, and bumps some of the other provider versions, the openlineage methods will fail. |
As discussed with @mobuchowski -> we need to add back-compatibility |
In OpenLineage/OpenLineage#2520 released in
openlineage-python==1.13.1
there were introduced V2 classes for facets and events.