Address PR comments

Author: fretz12

openapi/openapiv2.json

@@ -10453,10 +10453,6 @@
           "$ref": "#/definitions/v1Priority",
           "description": "Priority metadata."
         },
-        "startDelay": {
-          "type": "string",
-          "description": "Time to wait before dispatching the first activity."
-        },
         "completionCallbacks": {
           "type": "array",
           "items": {
@@ -10471,7 +10467,7 @@
             "type": "object",
             "$ref": "#/definitions/apiCommonV1Link"
           },
-          "description": "Links to be associated with the activity."
+          "description": "Links to be associated with the activity. Callbacks may also have associated links;\nlinks already included with a callback should not be duplicated here."
         }
       }
     },

openapi/openapiv3.yaml

@@ -14302,10 +14302,6 @@ components:
           allOf:
             - $ref: '#/components/schemas/Priority'
           description: Priority metadata.
-        startDelay:
-          pattern: ^-?(?:0|[1-9][0-9]{0,11})(?:\.[0-9]{1,9})?s$
-          type: string
-          description: Time to wait before dispatching the first activity.
         completionCallbacks:
           type: array
           items:
@@ -14317,7 +14313,9 @@ components:
           type: array
           items:
             $ref: '#/components/schemas/Link'
-          description: Links to be associated with the activity.
+          description: |-
+            Links to be associated with the activity. Callbacks may also have associated links;
+             links already included with a callback should not be duplicated here.
     StartActivityExecutionResponse:
       type: object
       properties:

temporal/api/workflowservice/v1/request_response.proto

@@ -2931,13 +2931,12 @@ message StartActivityExecutionRequest {
     temporal.api.sdk.v1.UserMetadata user_metadata = 17;
     // Priority metadata.
     temporal.api.common.v1.Priority priority = 18;
-    // Time to wait before dispatching the first activity.
-    google.protobuf.Duration start_delay = 19;
     // Callbacks to be called by the server when this activity reaches a terminal state.
     // Callback addresses must be whitelisted in the server's dynamic configuration.
-    repeated temporal.api.common.v1.Callback completion_callbacks = 20;
-    // Links to be associated with the activity.
-    repeated temporal.api.common.v1.Link links = 21;
+    repeated temporal.api.common.v1.Callback completion_callbacks = 19;
+    // Links to be associated with the activity. Callbacks may also have associated links;
+    // links already included with a callback should not be duplicated here.
+    repeated temporal.api.common.v1.Link links = 20;
 }
 
 message StartActivityExecutionResponse {