lukecwik commented on a change in pull request #11203: [BEAM-9577] Define and 
implement dependency-aware artifact staging service.
URL: https://github.com/apache/beam/pull/11203#discussion_r400604009
 
 

 ##########
 File path: model/job-management/src/main/proto/beam_artifact_api.proto
 ##########
 @@ -67,15 +70,10 @@ message ResolveArtifactRequest {
 
 // A response for artifact resolution.
 message ResolveArtifactResponse {
-  // A full set of replacements for the set of requested artifacts, preferably
-  // in terms of the requested type URNs.  If there is no better resolution,
-  // the original list is returned.
+  // A full (ordered) set of replacements for the set of requested artifacts,
 
 Review comment:
   ```suggestion
     // A fully (ordered) set of replacements for the set of requested 
artifacts,
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to