Skip to content

Commit 05e5047

Browse files
sean-zlaitechniq
andauthored
Add physicalType to NodeKey struct (#363)
## Summary Based on Slack [discussion](https://zipline-2kh4520.slack.com/archives/C0880ECQ0EN/p1739304132253249) ## Checklist - [ ] Added Unit Tests - [ ] Covered by existing CI - [ ] Integration tested - [ ] Documentation update <!-- av pr metadata This information is embedded by the av CLI when creating PRs to track the status of stacks when using Aviator. Please do not delete or edit this section of the PR. ``` {"parent":"main","parentHead":"","trunk":"main"} ``` --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced an optional attribute to enhance node classification with more detailed physical characteristics for improved metadata representation. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Sean Lynch <[email protected]>
1 parent d0064b2 commit 05e5047

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

api/thrift/orchestration.thrift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ struct NodeKey {
4242
1: optional string name
4343

4444
2: optional LogicalType logicalType
45+
3: optional PhysicalNodeType physicalType
4546

4647
/**
4748
* represents the computation of the node including the computation of all its parents

0 commit comments

Comments
 (0)