Skip to content

Conversation

@pedro93
Copy link
Collaborator

@pedro93 pedro93 commented Dec 19, 2025

Summary

Adds full SDK support for patching dataProcessInstanceInput and dataProcessInstanceOutput aspects in Python and Java SDKs.

Changes

Server-side (entity-registry)

  • DataProcessInstanceInputTemplate: Template for input/inputEdges array fields
  • DataProcessInstanceOutputTemplate: Template for output/outputEdges array fields
  • Registered in AspectTemplateEngine and SnapshotEntityRegistry

Java SDK patch builders (entity-registry)

  • DataProcessInstanceInputPatchBuilder: add/remove inputs and inputEdges
  • DataProcessInstanceOutputPatchBuilder: add/remove outputs and outputEdges
  • Unit tests for both builders

Java SDK v2 (datahub-client)

  • DataProcessInstance entity class with input/output lineage methods
  • Integration tests for input/output operations

Python SDK

  • DataProcessInstancePatchBuilder with add/remove methods for:
    • inputs (URN array)
    • inputEdges (Edge array)
    • outputs (URN array)
    • outputEdges (Edge array)
  • Unit tests for patch builder

Documentation

  • Updated docs/advanced/patch.md to include Data Process Instances in supported builders list

Smoke Tests

  • End-to-end tests for input/output/inputEdge/outputEdge operations

Test plan

  • Unit tests pass for DataProcessInstanceInputTemplateTest (4 tests)
  • Unit tests pass for DataProcessInstanceOutputTemplateTest (4 tests)
  • Unit tests pass for DataProcessInstanceInputPatchBuilderTest (4 tests)
  • Unit tests pass for DataProcessInstanceOutputPatchBuilderTest (4 tests)
  • Python unit tests pass for test_dataprocessinstance_patch.py (12 tests)
  • Smoke tests pass for test_dataprocessinstance_patches.py (12 tests)
  • Java SDK v2 integration tests pass (10 tests)
  • Code compiles successfully
  • Spotless/ruff formatting applied

🤖 Generated with Claude Code

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

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

Files with missing lines Patch % Lines
...cessinstance/DataProcessInstanceInputTemplate.java 90.47% 1 Missing and 1 partial ⚠️
...essinstance/DataProcessInstanceOutputTemplate.java 90.47% 1 Missing and 1 partial ⚠️
...tadata/models/registry/SnapshotEntityRegistry.java 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@pedro93 pedro93 force-pushed the feat/patch-dataprocessinstance-aspects branch from e4ba1b0 to 7918b16 Compare December 19, 2025 21:26
…spects

Adds SDK support for patching dataProcessInstanceInput and dataProcessInstanceOutput
aspects in both Python and Java SDKs.

## Server-side (entity-registry)
- DataProcessInstanceInputTemplate: Template for input/inputEdges array fields
- DataProcessInstanceOutputTemplate: Template for output/outputEdges array fields
- Registered in AspectTemplateEngine and SnapshotEntityRegistry

## Java SDK patch builders (entity-registry)
- DataProcessInstanceInputPatchBuilder: add/remove inputs and inputEdges
- DataProcessInstanceOutputPatchBuilder: add/remove outputs and outputEdges

## Java SDK v2 (datahub-client)
- DataProcessInstance entity class with input/output lineage methods
- Integration tests for input/output operations

## Python SDK
- DataProcessInstancePatchBuilder with add/remove methods for:
  - inputs (URN array)
  - inputEdges (Edge array)
  - outputs (URN array)
  - outputEdges (Edge array)

## Tests
- Java unit tests for templates and patch builders
- Python unit tests for patch builder
- Smoke tests for end-to-end validation

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@pedro93 pedro93 force-pushed the feat/patch-dataprocessinstance-aspects branch from 7918b16 to dd20b63 Compare December 19, 2025 21:31
@codecov
Copy link

codecov bot commented Dec 19, 2025

Bundle Report

Changes will increase total bundle size by 31.28MB (100.0%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
datahub-react-web-esm 31.28MB 31.28MB (100%) ⬆️⚠️

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.

2 participants