fix(ingest): fully support MCPs in urn_iter primitive #9157

hsheth2 · 2023-11-01T18:42:07Z

We previously would have an issue with urns nested with the MCP's aspect.

Fixes #9111.

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

metadata-ingestion/src/datahub/utilities/urns/urn_iter.py

fix(ingest): fully support MCPs in urn_iter primitive

a4928cd

github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Nov 1, 2023

hsheth2 requested a review from asikowitz November 1, 2023 18:46

asikowitz approved these changes Nov 1, 2023

View reviewed changes

metadata-ingestion/src/datahub/utilities/urns/urn_iter.py Outdated Show resolved Hide resolved

vercel bot deployed to Preview November 1, 2023 19:00 View deployment

tweak comment

f4a920b

hsheth2 added the merge-pending-ci A PR that has passed review and should be merged once CI is green. label Nov 1, 2023

vercel bot deployed to Preview November 1, 2023 20:30 View deployment

hsheth2 merged commit bab9d1c into datahub-project:master Nov 2, 2023
51 checks passed

hsheth2 deleted the urn-iter-fix branch November 2, 2023 03:47