[GitHub] [beam] lostluck commented on a change in pull request #16957: [BEAM-13829] Expose status API from Go SDK Harness

[GitBox <gi...@apache.org>]

lostluck commented on a change in pull request #16957:
URL: https://github.com/apache/beam/pull/16957#discussion_r819017568



##########
File path: sdks/go.mod
##########
@@ -23,33 +23,39 @@ module github.com/apache/beam/sdks/v2
 go 1.16
 
 require (
-	cloud.google.com/go/bigquery v1.17.0
-	cloud.google.com/go/datastore v1.5.0
-	cloud.google.com/go/pubsub v1.11.0-beta.schemas
-	cloud.google.com/go/storage v1.15.0
+	cloud.google.com/go/bigquery v1.28.0
+	cloud.google.com/go/compute v1.5.0 // indirect
+	cloud.google.com/go/datastore v1.6.0
+	cloud.google.com/go/iam v0.2.0 // indirect
+	cloud.google.com/go/pubsub v1.18.0
+	cloud.google.com/go/storage v1.21.0
+	github.com/bketelsen/crypt v0.0.4 // indirect

Review comment:
       Agreed that it's strange. Unless one is adding a new dependency/import it largely shouldn't show up here... strange. But I'm less worried about additions than I am about unintentional version updates.
   
   Note that with Go Modules, none of this will affect things for users unless they've got a conflicting requirement, at which point Go will pick the maximum of the two. This behavior tends to keep things up to date.  (Then one gets into the issue of breaking changes or bad versions, but I won't go into that here.)




-- 
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.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org