ibzib commented on a change in pull request #13791:
URL: https://github.com/apache/beam/pull/13791#discussion_r568196281
##########
File path: sdks/java/io/google-cloud-platform/build.gradle
##########
@@ -50,50 +53,53 @@ dependencies {
exclude group: 'io.grpc', module: 'grpc-grpclb'
}
compile library.java.google_cloud_core
- compile(library.java.google_cloud_core_grpc) {
Review comment:
I'm very hesitant about removing so many `compile` dependencies after we
discovered BEAM-11732. I think we should create a JIRA task to complete later,
add them all back, mark them as permitUnusedDeclared, and leave a comment
linking to the JIRA.
##########
File path: sdks/java/io/google-cloud-platform/expansion-service/build.gradle
##########
@@ -31,7 +32,5 @@ description = "Apache Beam :: SDKs :: Java :: IO :: Google
Cloud Platform :: Exp
ext.summary = "Expansion service serving GCP Java IOs"
dependencies {
- compile project(":sdks:java:expansion-service")
Review comment:
On second thought, let's just keep these
`compile`+`permitUnusedDeclared` to be safe.
##########
File path: sdks/java/io/google-cloud-platform/build.gradle
##########
@@ -50,50 +50,54 @@ dependencies {
exclude group: 'io.grpc', module: 'grpc-grpclb'
}
compile library.java.google_cloud_core
- compile(library.java.google_cloud_core_grpc) {
- exclude group: 'io.grpc', module: 'grpc-core' // Use Beam's version
- }
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_http_client
compile library.java.google_http_client_jackson2
- compile library.java.grpc_alts
compile library.java.grpc_auth
- compile library.java.grpc_core
- compile library.java.grpc_context
- compile library.java.grpc_grpclb
+ testImplementation 'io.grpc:grpc-core:1.27.2'
compile library.java.grpc_netty
- compile library.java.grpc_netty_shaded
- compile library.java.grpc_stub
compile library.java.grpc_google_cloud_pubsub_v1
- compile library.java.grpc_google_cloud_pubsublite_v1
compile library.java.guava
- compile library.java.hamcrest_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
- compile library.java.proto_google_cloud_bigquery_storage_v1beta1
compile library.java.proto_google_cloud_bigtable_v2
compile library.java.proto_google_cloud_datastore_v1
compile library.java.proto_google_cloud_pubsub_v1
compile library.java.proto_google_cloud_pubsublite_v1
- compile library.java.proto_google_cloud_spanner_admin_database_v1
compile library.java.proto_google_common_protos
compile library.java.protobuf_java
compile library.java.slf4j_api
+ compile "com.google.api.grpc:proto-google-cloud-bigquerystorage-v1:0.90.0"
Review comment:
Please add the "not available" dependencies to BeamModulePlugin so that
we can use the bom versions.
----------------------------------------------------------------
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]
