-
Notifications
You must be signed in to change notification settings - Fork 323
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
include column lineage in dataset resource #2148
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2148 +/- ##
=========================================
Coverage 76.33% 76.33%
Complexity 1099 1099
=========================================
Files 214 214
Lines 5139 5139
Branches 407 407
=========================================
Hits 3923 3923
Misses 762 762
Partials 454 454 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
70db39e
to
91d30f0
Compare
c65ecdd
to
791e1bf
Compare
91d30f0
to
5d49c51
Compare
6c35e81
to
8b669df
Compare
api/src/main/java/marquez/service/models/ColumnLineageInputField.java
Outdated
Show resolved
Hide resolved
8b669df
to
d4f0890
Compare
b79d712
to
8e66689
Compare
d4f0890
to
950e293
Compare
950e293
to
1ed1720
Compare
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.
Few minor nits, besides LGTM
1ed1720
to
2239520
Compare
Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>
2239520
to
b665ad6
Compare
Signed-off-by: Pawel Leszczynski leszczynski.pawel@gmail.com
Problem
Column lineage should be returned within
dataset
resource.Closes: #2113
Solution
ColumnLineageService
which enrichesDataset
with column lineage information,DatasetResource
.Checklist
CHANGELOG.md
with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary).sql
database schema migration according to Flyway's naming convention (if relevant)