New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deps: update dependency com.google.guava:guava-bom to v31 #498
deps: update dependency com.google.guava:guava-bom to v31 #498
Conversation
Checker-qual is causing a problem:
The message says Guava uses org.checkerframework:checker-qual:3.12.0. Other libraries use checker-qual 3.8.0. Does google-cloud-datacatalog or google-cloud-storage use checker-qual? No.It turned out that these libraries' checker-qual come from Guava's checker-qual dependency. Here are the dependency tree of google-cloud-datacatalog and its dependency proto-google-cloud-datacatalog-v1. https://gist.github.com/suztomo/4fb5595a1e357cc2958b1994884d27db#file-proto-google-cloud-datacatalog-v1-txt-L13 . Is it safe to ignore this checker-qual requireUpperBoundDeps failure? YesI think it's safe to ignore these failures because once the Google Cloud Java client libraries start using the new Guava version, which brings newer checker-qual dependency, their checker-qual declaration by the flatten-maven-plugin will get upgraded to the newer version. |
first-party-dependencies/pom.xml
Outdated
@@ -58,13 +58,13 @@ | |||
<grpc.version>1.41.0</grpc.version> | |||
<gax.version>2.5.0</gax.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting for GAX and Google Cloud Core.
"downstream / dependencies (11, java-pubsub)" and "downstream / dependencies (11, java-bigqueryconnection)" are complaining about
What was change?Before the new dependencies,
With new
It seems that the scope change happened in 2.0.4 https://search.maven.org/artifact/com.google.api/api-common/2.0.4/jar. I think the module that uses Why the scope matters?
And it also shows that a runtime-scope dependency of a compile-scope dependency has runtime scope in the project. Therefore, for proto-google-cloud-pubsub-v1 module, javax.annotation:javax.annotation-api:jar:1.3.2 was "runtime" scope, which is marked as not required for compilation. |
@@ -98,6 +98,7 @@ | |||
<groupId>javax.annotation</groupId> | |||
<artifactId>javax.annotation-api</artifactId> | |||
<version>${javax.annotations.version}</version> | |||
<scope>compile</scope> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<codec.version>1.15</codec.version> | ||
<httpcomponents.httpcore.version>4.4.14</httpcomponents.httpcore.version> | ||
<httpcomponents.httpclient.version>4.5.13</httpcomponents.httpclient.version> | ||
<!-- ensure checker-qual version matches what Guava uses --> | ||
<checker-qual.version>3.12.0</checker-qual.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Neenu1995 Declaring checker-qual seems to resolve the release-dependency problem between java-storage, java-datacatalog, and java-bigquery.
third-party-dependencies/pom.xml
Outdated
@@ -98,7 +98,6 @@ | |||
<groupId>javax.annotation</groupId> | |||
<artifactId>javax.annotation-api</artifactId> | |||
<version>${javax.annotations.version}</version> | |||
<scope>compile</scope> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without this explicit scope, it fails:
[INFO] -------------------------------------------------------------
Warning: COMPILATION WARNING :
[INFO] -------------------------------------------------------------
Warning: bootstrap class path not set in conjunction with -source 8
[INFO] 1 warning
[INFO] -------------------------------------------------------------
[INFO] -------------------------------------------------------------
Error: COMPILATION ERROR :
[INFO] -------------------------------------------------------------
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-spanner/proto-google-cloud-spanner-admin-instance-v1/src/main/java/com/google/spanner/admin/instance/v1/InstanceConfigName.java:[27,24] cannot find symbol
symbol: class Generated
location: package javax.annotation
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-spanner/proto-google-cloud-spanner-admin-instance-v1/src/main/java/com/google/spanner/admin/instance/v1/InstanceConfigName.java:[30,2] cannot find symbol
symbol: class Generated
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-spanner/proto-google-cloud-spanner-admin-instance-v1/src/main/java/com/google/spanner/admin/instance/v1/ProjectName.java:[27,24] cannot find symbol
symbol: class Generated
location: package javax.annotation
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-spanner/proto-google-cloud-spanner-admin-instance-v1/src/main/java/com/google/spanner/admin/instance/v1/ProjectName.java:[30,2] cannot find symbol
symbol: class Generated
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-spanner/proto-google-cloud-spanner-admin-instance-v1/src/main/java/com/google/spanner/admin/instance/v1/InstanceName.java:[27,24] cannot find symbol
symbol: class Generated
location: package javax.annotation
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-spanner/proto-google-cloud-spanner-admin-instance-v1/src/main/java/com/google/spanner/admin/instance/v1/InstanceName.java:[30,2] cannot find symbol
symbol: class Generated
[INFO] 6 errors
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Google Cloud Spanner Parent 6.13.0:
[INFO]
[INFO] Google Cloud Spanner Parent ........................ SUCCESS [ 2.481 s]
[INFO] proto-google-cloud-spanner-admin-instance-v1 ....... FAILURE [ 2.380 s]
[INFO] proto-google-cloud-spanner-v1 ...................... SKIPPED
[INFO] proto-google-cloud-spanner-admin-database-v1 ....... SKIPPED
[INFO] grpc-google-cloud-spanner-v1 ....................... SKIPPED
[INFO] grpc-google-cloud-spanner-admin-instance-v1 ........ SKIPPED
[INFO] grpc-google-cloud-spanner-admin-database-v1 ........ SKIPPED
[INFO] Google Cloud Spanner ............................... SKIPPED
[INFO] Google Cloud Spanner BOM ........................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 5.566 s
[INFO] Finished at: 2021-10-15T21:21:53Z
[INFO] ------------------------------------------------------------------------
first-party-dependencies/pom.xml
Outdated
@@ -56,7 +56,7 @@ | |||
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> | |||
<site.installationModule>${project.artifactId}</site.installationModule> | |||
<grpc.version>1.41.0</grpc.version> | |||
<gax.version>2.5.0</gax.version> | |||
<gax.version>2.6.0</gax.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now this version fails with missing com.google.auth.oauth2
package.
[INFO] -------------------------------------------------------------
Error: COMPILATION ERROR :
[INFO] -------------------------------------------------------------
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-pubsub/google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/Publisher.java:[43,30] package com.google.auth.oauth2 does not exist
Error: /home/runner/work/java-shared-dependencies/java-shared-dependencies/java-pubsub/google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/Subscriber.java:[37,30] package com.google.auth.oauth2 does not exist
[INFO] 2 errors
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Google Cloud Pub/Sub Parent 1.114.7-SNAPSHOT ....... SUCCESS [ 2.460 s]
[INFO] proto-google-cloud-pubsub-v1 1.96.7-SNAPSHOT ....... SUCCESS [ 1.522 s]
[INFO] grpc-google-cloud-pubsub-v1 1.96.7-SNAPSHOT ........ SUCCESS [ 0.905 s]
[INFO] Google Cloud Pub/Sub 1.114.7-SNAPSHOT .............. FAILURE [ 1.644 s]
[INFO] Google Cloud pubsub BOM 1.114.7-SNAPSHOT ........... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
@@ -66,7 +66,7 @@ | |||
<google.auth.version>1.2.1</google.auth.version> | |||
<google.http-client.version>1.40.1</google.http-client.version> | |||
<google.oauth-client.version>1.32.1</google.oauth-client.version> | |||
<google.api-client.version>1.32.1</google.api-client.version> | |||
<google.api-client.version>1.32.2</google.api-client.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you.
dba11b5
into
googleapis:main
🤖 I have created a release \*beep\* \*boop\* --- ## [2.4.0](https://www.github.com/googleapis/java-shared-dependencies/compare/2.3.0...v2.4.0) (2021-10-18) ### Dependencies * update dependency com.fasterxml.jackson:jackson-bom to v2.13.0 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.api-client:google-api-client-bom to v1.32.2 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.api:api-common to v2.0.5 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.auth:google-auth-library-bom to v1.2.1 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.guava:guava-bom to v31 ([#498](https://www.github.com/googleapis/java-shared-dependencies/issues/498)) ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.http-client:google-http-client-bom to v1.40.1 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.protobuf:protobuf-bom to v3.18.1 ([#494](https://www.github.com/googleapis/java-shared-dependencies/issues/494)) ([a30e376](https://www.github.com/googleapis/java-shared-dependencies/commit/a30e3762b5fb0e2d31304d211c12eb2c1320f0c1)) * update dependency io.grpc:grpc-bom to v1.41.0 ([#482](https://www.github.com/googleapis/java-shared-dependencies/issues/482)) ([8a7d227](https://www.github.com/googleapis/java-shared-dependencies/commit/8a7d227f2a1dab9ff23d247a4e6131c817804c36)) * update gax.version to v2.6.1 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update iam.version to v1.1.6 ([#485](https://www.github.com/googleapis/java-shared-dependencies/issues/485)) ([cef7e09](https://www.github.com/googleapis/java-shared-dependencies/commit/cef7e090bbc2e3ae775fa83505f98df286ba7d42)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
…#498) deps: update gax.version to v2.6.1 deps: update dependency com.google.api-client:google-api-client-bom to v1.32.2 deps: update dependency com.google.http-client:google-http-client-bom to v1.40.1 deps: update dependency com.google.api:api-common to v2.0.5 deps: update dependency com.google.auth:google-auth-library-bom to v1.2.1 deps: update dependency com.fasterxml.jackson:jackson-bom to v2.13.0
🤖 I have created a release \*beep\* \*boop\* --- ## [2.4.0](https://www.github.com/googleapis/java-shared-dependencies/compare/2.3.0...v2.4.0) (2021-10-18) ### Dependencies * update dependency com.fasterxml.jackson:jackson-bom to v2.13.0 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.api-client:google-api-client-bom to v1.32.2 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.api:api-common to v2.0.5 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.auth:google-auth-library-bom to v1.2.1 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.guava:guava-bom to v31 ([googleapis#498](https://www.github.com/googleapis/java-shared-dependencies/issues/498)) ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.http-client:google-http-client-bom to v1.40.1 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update dependency com.google.protobuf:protobuf-bom to v3.18.1 ([googleapis#494](https://www.github.com/googleapis/java-shared-dependencies/issues/494)) ([a30e376](https://www.github.com/googleapis/java-shared-dependencies/commit/a30e3762b5fb0e2d31304d211c12eb2c1320f0c1)) * update dependency io.grpc:grpc-bom to v1.41.0 ([googleapis#482](https://www.github.com/googleapis/java-shared-dependencies/issues/482)) ([8a7d227](https://www.github.com/googleapis/java-shared-dependencies/commit/8a7d227f2a1dab9ff23d247a4e6131c817804c36)) * update gax.version to v2.6.1 ([dba11b5](https://www.github.com/googleapis/java-shared-dependencies/commit/dba11b5fe14d39cdd62357e7bb1b82831c481150)) * update iam.version to v1.1.6 ([googleapis#485](https://www.github.com/googleapis/java-shared-dependencies/issues/485)) ([cef7e09](https://www.github.com/googleapis/java-shared-dependencies/commit/cef7e090bbc2e3ae775fa83505f98df286ba7d42)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
deps: update gax.version to v2.6.1
deps: update dependency com.google.api-client:google-api-client-bom to v1.32.2
deps: update dependency com.google.http-client:google-http-client-bom to v1.40.1
deps: update dependency com.google.api:api-common to v2.0.5
deps: update dependency com.google.auth:google-auth-library-bom to v1.2.1
deps: update dependency com.fasterxml.jackson:jackson-bom to v2.13.0