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

vdk-lineage: introducing POC (pre-alpha) implementation #783

Merged
merged 2 commits into from
Apr 14, 2022

Conversation

antoniivanov
Copy link
Collaborator

VDK Lineage plugin provides lineage data (input data -> job -> output
data) information and send it to pre-configured
destination.

At POC level currently (Pre-Alpha release).
It collects lineage information for each job run and for each executed
query.
Query execution is currently before it's executed (so not query status
is logged) as we lack the necessary hooks.

There are plenty of TODOs in the code that need to be addressed before
moving maturity level and add more tests.

Testing Done: there are some tests and also manually:

vdk marquez-server --start
vdk run some-job
vdk marquez-server --stop

and saw lineage in marquz ui

Signed-off-by: Antoni Ivanov [email protected]

@antoniivanov antoniivanov force-pushed the person/aivanov/vdk-lineage branch 4 times, most recently from 47fc17b to 1eec655 Compare April 6, 2022 09:03
Copy link
Contributor

@doks5 doks5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

VDK Lineage plugin provides lineage data (input data -> job -> output
data) information and send it to pre-configured
destination.

At POC level currently (Pre-Alpha release).
It collects lineage information for each job run and for each executed
query.
Query execution is currently before it's executed (so not query status
is logged) as we lack the necessary hooks.

There are plenty of TODOs in the code that need to be addressed before
moving maturity level and add more tests.

Testing Done: there are some tests and also manually:
```
vdk marquez-server --start
vdk run some-job
vdk marquez-server --stop
```

Signed-off-by: Antoni Ivanov <[email protected]>
@antoniivanov antoniivanov force-pushed the person/aivanov/vdk-lineage branch from 1eec655 to 521082d Compare April 14, 2022 12:51
@antoniivanov antoniivanov merged commit 36b2782 into main Apr 14, 2022
@antoniivanov antoniivanov deleted the person/aivanov/vdk-lineage branch April 14, 2022 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants