Skip to content
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 spanner to 3.1.2 #306

Merged
merged 1 commit into from Dec 16, 2020
Merged

deps: update spanner to 3.1.2 #306

merged 1 commit into from Dec 16, 2020

Conversation

olavloite
Copy link
Collaborator

(Temporarily) fixes the dependency problem caused by the addition of grpc-alts as a runtime dependency in google-cloud-spanner and updates Spanner client library to 3.1.2.

Once googleapis/java-spanner#757 has been merged and is available in the shared libraries, or another fix for the problem described in that PR, the TODO's in this PR can be reverted.

Replaces #293

@olavloite olavloite requested review from a team as code owners December 16, 2020 12:09
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/java-spanner-jdbc API. label Dec 16, 2020
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Dec 16, 2020
@codecov
Copy link

codecov bot commented Dec 16, 2020

Codecov Report

Merging #306 (e610f5b) into master (8e01a57) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #306   +/-   ##
=========================================
  Coverage     70.90%   70.90%           
  Complexity     1104     1104           
=========================================
  Files            24       24           
  Lines          3420     3420           
  Branches        525      525           
=========================================
  Hits           2425     2425           
  Misses          780      780           
  Partials        215      215           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e01a57...e610f5b. Read the comment docs.

@@ -145,10 +150,6 @@
<groupId>com.google.api.grpc</groupId>
<artifactId>proto-google-cloud-spanner-v1</artifactId>
</dependency>
<dependency>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a duplicate of the dependency right above.

@thiagotnunes thiagotnunes merged commit 596e8ed into master Dec 16, 2020
@thiagotnunes thiagotnunes deleted the deps-spanner-3_1_2 branch December 16, 2020 23:16
gcf-merge-on-green bot pushed a commit that referenced this pull request Dec 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the googleapis/java-spanner-jdbc API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants