From 33a7d4d3362fa83108a85e1ca8bfd10ccedd2e40 Mon Sep 17 00:00:00 2001
From: Chen Sun <chensun@users.noreply.github.com>
Date: Wed, 17 Feb 2021 12:20:30 -0800
Subject: [PATCH] feat(api): Update pipeline_spec.proto - Add metadata to
 ArtifactSpec (#5143)

---
 api/v2alpha1/pipeline_spec.proto | 32 ++++++++++++++++++++++++--------
 1 file changed, 24 insertions(+), 8 deletions(-)

diff --git a/api/v2alpha1/pipeline_spec.proto b/api/v2alpha1/pipeline_spec.proto
index 869619f525..86837e2a90 100644
--- a/api/v2alpha1/pipeline_spec.proto
+++ b/api/v2alpha1/pipeline_spec.proto
@@ -1,11 +1,12 @@
 syntax = "proto3";
 
-option go_package = "github.com/kubeflow/pipelines/api/v2alpha1/go";
 package ml_pipelines;
 
 import "google/protobuf/any.proto";
 import "google/protobuf/struct.proto";
 
+option go_package = "github.com/kubeflow/pipelines/api/v2alpha1/go";
+
 // The spec of a pipeline job.
 message PipelineJob {
   string name = 1;  // Name of the job.
@@ -198,8 +199,13 @@ message ComponentOutputsSpec {
   // Definition of an artifact output.
   message ArtifactSpec {
     ArtifactTypeSchema artifact_type = 1;
-    map<string, ValueOrRuntimeParameter> properties = 2;
-    map<string, ValueOrRuntimeParameter> custom_properties = 3;
+    // Deprecated. Use [ArtifactSpec.metadata][] instead.
+    map<string, ValueOrRuntimeParameter> properties = 2 [deprecated = true];
+    // Deprecated. Use [ArtifactSpec.metadata][] instead.
+    map<string, ValueOrRuntimeParameter> custom_properties = 3
+        [deprecated = true];
+    // Properties of the Artifact.
+    google.protobuf.Struct metadata = 4;
   }
   // Definition of a parameter output.
   message ParameterSpec {
@@ -581,10 +587,16 @@ message PipelineDeploymentConfig {
     ArtifactTypeSchema type_schema = 2;
 
     // The properties of the artifact.
-    map<string, ValueOrRuntimeParameter> properties = 3;
+    // Deprecated. Use [ImporterSpec.metadata][] instead.
+    map<string, ValueOrRuntimeParameter> properties = 3 [deprecated = true];
 
     // The custom properties of the artifact.
-    map<string, ValueOrRuntimeParameter> custom_properties = 4;
+    // Deprecated. Use [ImporterSpec.metadata][] instead.
+    map<string, ValueOrRuntimeParameter> custom_properties = 4
+        [deprecated = true];
+
+    // Properties of the Artifact.
+    google.protobuf.Struct metadata = 6;
 
     // Whether or not import an artifact regardless it has been imported before.
     bool reimport = 5;
@@ -673,10 +685,15 @@ message RuntimeArtifact {
   string uri = 3;
 
   // The properties of the artifact.
-  map<string, Value> properties = 4;
+  // Deprecated. Use [RuntimeArtifact.metadata][] instead.
+  map<string, Value> properties = 4 [deprecated = true];
 
   // The custom properties of the artifact.
-  map<string, Value> custom_properties = 5;
+  // Deprecated. Use [RuntimeArtifact.metadata][] instead.
+  map<string, Value> custom_properties = 5 [deprecated = true];
+
+  // Properties of the Artifact.
+  google.protobuf.Struct metadata = 6;
 }
 
 // Message that represents a list of artifacts.
@@ -711,7 +728,6 @@ message ArtifactList {
 // 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 {
-
   // The runtime inputs data of the execution.
   message Inputs {
     // Input parameters of the execution.