Skip to content

Commit

Permalink
Update IR ExecutorOutput message to use ArtifactList for artifacts fi…
Browse files Browse the repository at this point in the history
…eld. (#4619)

* Update IR ExecutorOutput message

* fix a typo
  • Loading branch information
chensun authored Oct 13, 2020
1 parent f445c09 commit 72ce138
Show file tree
Hide file tree
Showing 2 changed files with 42 additions and 41 deletions.
13 changes: 7 additions & 6 deletions api/v2alpha1/pipeline_spec.proto
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,12 @@ message RuntimeArtifact {
map<string, Value> custom_properties = 5;
}

// Message that represents a list of artifacts.
message ArtifactList {
// A list of artifacts.
repeated RuntimeArtifact artifacts = 1;
}

// The input of an executor, which includes all the data that
// can be passed into the executor spec by a string based placeholder.
//
Expand All @@ -364,11 +370,6 @@ message RuntimeArtifact {
// output file and executor output metadata files are set by the container, the
// output metadata file will have higher precedence to set output parameters.
message ExecutorInput {
// Message that represents a list of artifacts.
message ArtifactList {
// A list of parameters.
repeated RuntimeArtifact artifacts = 1;
}

// The runtime inputs data of the execution.
message Inputs {
Expand Down Expand Up @@ -416,5 +417,5 @@ message ExecutorOutput {
map<string, Value> parameters = 1;

// The updated metadata for output artifact.
map<string, RuntimeArtifact> artifacts = 2;
map<string, ArtifactList> artifacts = 2;
}
Loading

0 comments on commit 72ce138

Please sign in to comment.