Skip to content

fix(cli): remove unused sql and errorMessage OpenLineage facets - #2391

Merged
rafal-hawrylak merged 1 commit into
mainfrom
fix-cli-lineage-remove-unused-facets
Oct 20, 2026
Merged

rafal-hawrylak merged 1 commit into
mainfrom
fix-cli-lineage-remove-unused-facets

Conversation

@rafal-hawrylak

Copy link
Copy Markdown
Collaborator

Summary

  • Remove job.facets.sql (SQLJobFacet) and run.facets.errorMessage (ErrorMessageRunFacet) from LineagePayloadBuilder.
  • The Data Lineage API ProcessOpenLineageRunEvent endpoint does not ingest sql or errorMessage facets; Knowledge Catalog resolves the executed SQL query and job error status directly from BigQuery via run.facets.externalQuery.
  • Removing these unused facets avoids sending raw SQL statements and task error messages in OpenLineage payloads and aligns the CLI payload structure with Cloud Dataform.

Test Plan

  • ./scripts/lint
  • ./scripts/run_tests
  • ./scripts/run_integration_tests

The Data Lineage API ProcessOpenLineageRunEvent endpoint does not ingest job.facets.sql (SQLJobFacet) or run.facets.errorMessage (ErrorMessageRunFacet); Knowledge Catalog instead resolves the executed SQL query and job status from BigQuery via run.facets.externalQuery. Removing these unused facets avoids sending raw SQL text and task error messages in OpenLineage payloads and aligns the CLI payload builder with Cloud Dataform.
@rafal-hawrylak
rafal-hawrylak requested a review from a team as a code owner October 20, 2026 08:15
@rafal-hawrylak
rafal-hawrylak requested review from apilaskowski and removed request for a team October 20, 2026 08:15
@rafal-hawrylak
rafal-hawrylak merged commit 471674a into main Oct 20, 2026
14 checks passed
@rafal-hawrylak
rafal-hawrylak deleted the fix-cli-lineage-remove-unused-facets branch October 20, 2026 12:14
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.

3 participants