You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2021/02/25 21:43:06 UTC

[GitHub] [beam] shehzaadn-vd commented on a change in pull request #13791: [BEAM-10961] Strict dependency checking for sdks/java/io/gcp

shehzaadn-vd commented on a change in pull request #13791:
URL: https://github.com/apache/beam/pull/13791#discussion_r583222291



##########
File path: sdks/java/io/google-cloud-platform/build.gradle
##########
@@ -53,48 +58,78 @@ dependencies {
   compile(library.java.google_cloud_core_grpc) {
     exclude group: 'io.grpc', module: 'grpc-core' // Use Beam's version
   }
+  permitUnusedDeclared library.java.google_cloud_core_grpc // BEAM-11761
   compile library.java.google_cloud_datastore_v1_proto_client
   compile library.java.google_cloud_pubsublite
   // GCP PubSub client is used in TestPubSub
   compile library.java.google_cloud_pubsub
   compile library.java.google_cloud_spanner
+  compile library.java.google_code_gson
   compile library.java.google_http_client
   compile library.java.google_http_client_jackson2
   compile library.java.grpc_alts
+  permitUnusedDeclared library.java.grpc_alts // BEAM-11761
   compile library.java.grpc_api
   compile library.java.grpc_auth
   compile library.java.grpc_core
+  permitUnusedDeclared library.java.grpc_core // BEAM-11761
   compile library.java.grpc_context
+  permitUnusedDeclared library.java.grpc_context // BEAM-11761
   compile library.java.grpc_grpclb
+  permitUnusedDeclared library.java.grpc_grpclb // BEAM-11761
   compile library.java.grpc_netty
   compile library.java.grpc_netty_shaded
+  permitUnusedDeclared library.java.grpc_netty_shaded // BEAM-11761
   compile library.java.grpc_stub
+  permitUnusedDeclared library.java.grpc_stub // BEAM-11761
   compile library.java.grpc_google_cloud_pubsub_v1
   compile library.java.grpc_google_cloud_pubsublite_v1
+  permitUnusedDeclared library.java.grpc_google_cloud_pubsublite_v1 // BEAM-11761
   compile library.java.guava
+  compile library.java.hamcrest
+  compile library.java.http_client
   compile library.java.hamcrest_core
+  permitUnusedDeclared library.java.hamcrest_core // BEAM-11761
+  compile library.java.http_core
+  compile library.java.jackson_core
   compile library.java.jackson_databind
   compile library.java.joda_time
   compile library.java.junit
   compile library.java.netty_handler
   compile library.java.netty_tcnative_boringssl_static
+  permitUnusedDeclared library.java.netty_tcnative_boringssl_static // BEAM-11761
+  compile library.java.proto_google_cloud_bigquery_storage_v1
+  compile library.java.proto_google_cloud_bigquery_storage_v1beta1

Review comment:
       Ah yes. Should have noticed that after I merged with master.




----------------------------------------------------------------
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:
users@infra.apache.org