Skip to content

feat(ingest: add modifiers on capability on sources #13954

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

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

anshbansal
Copy link
Collaborator

@anshbansal anshbansal commented Jul 3, 2025

  • Merged docgen.sh into build.gradle
  • Add SourceCapabilityModifier in subtypes.py which is a Union of all other subtypes
  • Add subtype_modifier in capability annotation so we can have the sub type modifiers available programatically. This allows us to say we support coarse lineage for tables and stored procedures rather than just saying we support coarse lineage for the source.
  • Update capability summary generation to add those subtype modifiers in the generated json file so it is available on frontend.
  • Use the subtype modifiers to explicitly call out what types we support it for.

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Jul 3, 2025
Copy link

codecov bot commented Jul 3, 2025

❌ Unsupported file format

Upload processing failed due to unsupported file format. Please review the parser error message:
Error parsing JUnit XML in /home/runner/work/datahub/datahub/metadata-io/build/test-results/test/TEST-com.linkedin.metadata.graph.search.opensearch.SearchGraphServiceOpenSearchTest.xml at 117:1052

Caused by:
RuntimeError: Error converting computed name to ValidatedString

Caused by:
    string is too long</code></pre>

For more help, visit our troubleshooting guide.

@anshbansal anshbansal changed the title update capability summary feat(ingest: add modifiers on capability on sources Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ingestion PR or Issue related to the ingestion of metadata
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant