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.

Sign up for GitHub

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

Jump to bottom

feat(ingestion/transformer): Handle overlapping while mapping in extract ownership from tags transformer #10201

Merged
merged 14 commits into from
Apr 4, 2024
Merged

feat(ingestion/transformer): Handle overlapping while mapping in extract ownership from tags transformer #10201

merged 14 commits into from
Apr 4, 2024

Conversation

shubhamjagtap639
Copy link
Contributor

Checklist

  • The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
  • For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

shubhamjagtap639 and others added 13 commits March 14, 2024 13:09
@shubhamjagtap639
Add support of BigQuery Owner Label to Datahub Owner ingestion
b026e87
@shubhamjagtap639
Address review comments
1bbace7
@shubhamjagtap639
Revert bigquery owner label changes
077ee7c
@shubhamjagtap639
Add capture_owners_from_tags transformer
7c3e818
@shubhamjagtap639
Remove new capture_owners_from_tags and instead extended old extract_…
863651f 
…ownership_from_tags
@shubhamjagtap639
Address review comments
2306086
@shubhamjagtap639
Modify test case data
4beb576
@shubhamjagtap639
Address review comments
5bbff7b
@shubhamjagtap639
Merge branch 'master' of https://github.com/shubhamjagtap639/datahub …
3c12f6d 
…into BigQuery-Owner-Label-to-Datahub-Ownership
@shubhamjagtap639
Address review comments
d0cec72
@shubhamjagtap639
Merge branch 'datahub-project:master' into BigQuery-Owner-Label-to-Da…
16ecd49 
…tahub-Ownership
@shubhamjagtap639
Add code to handle overlapping replacement while converting tag as pe…
cadb489 
…r mapping
@shubhamjagtap639
Update the dataset transformer doc
bb2e243
@shubhamjagtap639 shubhamjagtap639 changed the title feat(ingestion/transformer): Handle overlapping replacement while mapping in extract ownership from tags transformer feat(ingestion/transformer): Handle overlapping while mapping in extract ownership from tags transformer Apr 4, 2024
@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Apr 4, 2024
@shubhamjagtap639 shubhamjagtap639 marked this pull request as ready for review April 4, 2024 17:52
@hsheth2 hsheth2 merged commit fa139a5 into datahub-project:master Apr 4, 2024
56 of 57 checks passed
sleeperdeep pushed a commit to sleeperdeep/datahub that referenced this pull request Jun 25, 2024
@shubhamjagtap639 @sleeperdeep
feat(ingestion/transformer): Handle overlapping while mapping in extr…
3f2ddac 
…act ownership from tags transformer (datahub-project#10201)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hsheth2 hsheth2 hsheth2 approved these changes

Assignees
No one assigned
Labels
ingestion PR or Issue related to the ingestion of metadata
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shubhamjagtap639 @hsheth2