-
Notifications
You must be signed in to change notification settings - Fork 310
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
API module # use OpenLineage model in OL resource #1593
Conversation
Signed-off-by: olek <oleksandr.dvornik@gmail.com>
Signed-off-by: olek <oleksandr.dvornik@gmail.com>
Signed-off-by: olek <oleksandr.dvornik@gmail.com>
Codecov Report
@@ Coverage Diff @@
## main #1593 +/- ##
============================================
- Coverage 78.63% 78.59% -0.05%
+ Complexity 959 957 -2
============================================
Files 188 187 -1
Lines 5191 5134 -57
Branches 415 408 -7
============================================
- Hits 4082 4035 -47
+ Misses 670 660 -10
Partials 439 439
Continue to review full report at Codecov.
|
Signed-off-by: olek <oleksandr.dvornik@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@OleksandrDvornik: just as a reminder, please link related issues in your PR descriptions, see our |
This reverts commit 1a96320. Signed-off-by: Michael Collado <mike@datakin.com>
This reverts commit 1a96320. Signed-off-by: Michael Collado <mike@datakin.com>
* Revert "API module # use OpenLineage model in OL resource (#1593)" This reverts commit 1a96320. Signed-off-by: Michael Collado <mike@datakin.com> * Fix Spark openlineage package names in Spark client Signed-off-by: Michael Collado <mike@datakin.com> * Updated LineageEvent to be forward compatible with OpenLineage RunEvents Signed-off-by: Michael Collado <mike@datakin.com>
PR for issue
Replace Marquez LineageEvent with OpenLineage model