Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename TaskParameter -> Input and Output #86

Merged
merged 1 commit into from
Nov 10, 2017
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 36 additions & 5 deletions task_execution.proto
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,13 @@ message Task {
//
// Input files.
// Inputs will be downloaded and mounted into the executor container.
repeated TaskParameter inputs = 6;
repeated Input inputs = 6;

// OPTIONAL
//
// Output files.
// Outputs will be uploaded from the executor container to long-term storage.
repeated TaskParameter outputs = 7;
repeated Output outputs = 7;

// OPTIONAL
//
Expand Down Expand Up @@ -76,8 +76,8 @@ enum FileType {
DIRECTORY = 1;
}

// TaskParameter describes input and output files for a Task.
message TaskParameter {
// Input describes Task input files.
message Input {

// OPTIONAL
string name = 1;
Expand Down Expand Up @@ -116,6 +116,37 @@ message TaskParameter {
string content = 6;
}

// Output describes Task output files.
message Output {

// OPTIONAL
string name = 1;

// OPTIONAL
string description = 2;

// REQUIRED
//
// URL in long term storage, for example:
// s3://my-object-store/file1
// gs://my-bucket/file2
// file:///path/to/my/file
// /path/to/my/file
// etc...
string url = 3;

// REQUIRED
//
// Path of the file inside the container.
// Must be an absolute path.
string path = 4;

// REQUIRED
//
// Type of the file, FILE or DIRECTORY
FileType type = 5;
}

// Ports describes the port binding between the container and host.
// For example, a Docker implementation might map this to `docker run -p host:container`.
message Ports {
Expand Down Expand Up @@ -412,7 +443,7 @@ enum TaskView {
// Task message will include all fields EXCEPT:
// Task.ExecutorLog.stdout
// Task.ExecutorLog.stderr
// TaskParameter.content in Task.Inputs
// Input.content
BASIC = 1;

// Task message includes all fields.
Expand Down