Skip to content

Conversation

@acrylJonny
Copy link
Collaborator

@acrylJonny acrylJonny commented Nov 10, 2025

Add Configurable Environment and Orchestrator for OpenLineage Integration
Adding flexible configuration options for the DataHub OpenLineage REST endpoint, allowing users to customize the environment/fabric type and orchestrator without requiring code changes or modifying OpenLineage events.

Motivation
DataHub's OpenLineage endpoint previously had hardcoded values:
Fabric Type: Always set to PROD
Orchestrator: Tightly coupled to either the processing_engine facet or producer URL parsing

This limitation prevented users from:
Ingesting metadata from multiple environments (DEV, TEST, QA, STG, etc.) into a single DataHub instance
Using custom orchestrators or metadata management tools (Unity Catalog, Dataplex, DataWorks, custom platforms) that don't match known patterns

@github-actions github-actions bot added ingestion PR or Issue related to the ingestion of metadata docs Issues and Improvements to docs devops PR or Issue related to DataHub backend & deployment labels Nov 10, 2025
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Nov 10, 2025
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

❌ Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...i/openlineage/config/OpenLineageServletConfig.java 88.23% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@datahub-cyborg datahub-cyborg bot added pending-submitter-response Issue/request has been reviewed but requires a response from the submitter and removed needs-review Label for PRs that need review from a maintainer. labels Nov 11, 2025
@datahub-cyborg datahub-cyborg bot added needs-review Label for PRs that need review from a maintainer. and removed pending-submitter-response Issue/request has been reviewed but requires a response from the submitter labels Nov 11, 2025
@datahub-cyborg datahub-cyborg bot added pending-submitter-response Issue/request has been reviewed but requires a response from the submitter and removed needs-review Label for PRs that need review from a maintainer. labels Nov 11, 2025
@datahub-cyborg datahub-cyborg bot added needs-review Label for PRs that need review from a maintainer. and removed pending-submitter-response Issue/request has been reviewed but requires a response from the submitter labels Nov 11, 2025
@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Nov 11, 2025
@acrylJonny acrylJonny enabled auto-merge (squash) November 11, 2025 18:34
@acrylJonny acrylJonny merged commit db65ea5 into master Nov 11, 2025
61 of 74 checks passed
@acrylJonny acrylJonny deleted the openlineage-improvements-nov branch November 11, 2025 19:01
esteban pushed a commit that referenced this pull request Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops PR or Issue related to DataHub backend & deployment docs Issues and Improvements to docs ingestion PR or Issue related to the ingestion of metadata pending-submitter-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants