Skip to content

generate lineage - #160

Merged
joroscoSF merged 6 commits into
mainfrom
jo_lineage_gen
Oct 16, 2026
Merged

joroscoSF merged 6 commits into
mainfrom
jo_lineage_gen

Conversation

@joroscoSF

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/datacustomcode/lineage/capture.py Outdated
Comment thread src/datacustomcode/client.py Outdated

Only the columns that feed a column's *value* are sources. Filter predicates,
join conditions and sort keys gate or order rows and are deliberately ignored.
"""

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a lot of code- can we explore integrating with something like OpenLineage to take care of this part for us?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks again for the research on this!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

👍

Comment thread README.md

### Generating lineage

`generate_lineage` runs a script entry point locally and derives the field-level `dataLineage` section of `config.json` from the Spark plans of the DataFrames it writes:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we speak more to this tool being a good "starting point to review" or however we want to phrase it - and how accurate, full data lineage is the author's responsibility?

markdlv-sf
markdlv-sf previously approved these changes Oct 15, 2026
@joroscoSF
joroscoSF merged commit d94ad4b into main Oct 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants