Skip to content

Commit

Permalink
Add version control for ArrayNode eventing (#4165)
Browse files Browse the repository at this point in the history
* added version control of ArrayNode eventing

Signed-off-by: Daniel Rammer <[email protected]>

* correctly sending initializing status for subtasks

Signed-off-by: Daniel Rammer <[email protected]>

* handling abort eventing

Signed-off-by: Daniel Rammer <[email protected]>

* fixed unit tests

Signed-off-by: Daniel Rammer <[email protected]>

* fixed lint issues

Signed-off-by: Daniel Rammer <[email protected]>

* adding IsArray flag to NodeExecutionMetadata

Signed-off-by: Daniel Rammer <[email protected]>

* fixed propeller generate

Signed-off-by: Daniel Rammer <[email protected]>

* fixed merging issues

Signed-off-by: Daniel Rammer <[email protected]>

* make generate for arraynode mocks

Signed-off-by: Daniel Rammer <[email protected]>

* removed dead code

Signed-off-by: Daniel Rammer <[email protected]>

---------

Signed-off-by: Daniel Rammer <[email protected]>
  • Loading branch information
hamersaw authored Oct 9, 2023
1 parent 8d66d05 commit 4a780c3
Show file tree
Hide file tree
Showing 40 changed files with 1,429 additions and 668 deletions.
4 changes: 4 additions & 0 deletions flyteadmin/pkg/repositories/transformers/node_execution.go
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ func CreateNodeExecutionModel(ctx context.Context, input ToNodeExecutionModelInp
SpecNodeId: input.Request.Event.SpecNodeId,
IsParentNode: input.Request.Event.IsParent,
IsDynamic: input.Request.Event.IsDynamic,
IsArray: input.Request.Event.IsArray,
}
err := handleNodeExecutionInputs(ctx, nodeExecution, input.Request, input.StorageClient)
if err != nil {
Expand Down Expand Up @@ -304,6 +305,9 @@ func UpdateNodeExecutionModel(
if request.Event.IsDynamic {
nodeExecutionMetadata.IsDynamic = true
}
if request.Event.IsArray {
nodeExecutionMetadata.IsArray = true
}
nodeExecMetadataBytes, err := proto.Marshal(&nodeExecutionMetadata)
if err != nil {
return errors.NewFlyteAdminErrorf(codes.Internal,
Expand Down
158 changes: 100 additions & 58 deletions flyteidl/gen/pb-cpp/flyteidl/admin/node_execution.pb.cc

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

21 changes: 21 additions & 0 deletions flyteidl/gen/pb-cpp/flyteidl/admin/node_execution.pb.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 4a780c3

Please sign in to comment.