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

feat: expose more methods from Connection in JDBC #255

Merged
merged 4 commits into from Dec 5, 2020

Conversation

olavloite
Copy link
Collaborator

More methods from the Connection API should be exposed in the Cloud Spanner JDBC Connection interface to make it easier to execute read-only transactions with specific timestamp bounds.

This PR relies on this PR in the client library that is needed before this can be merged.

Towards #253

@olavloite olavloite added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 4, 2020
@olavloite olavloite requested a review from a team as a code owner November 4, 2020 15:08
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 4, 2020
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/java-spanner-jdbc API. label Nov 4, 2020
More methods from the Connection API should be exposed in the
Cloud Spanner JDBC Connection interface to make it easier to
execute read-only transactions with specific timestamp bounds.

Towards #253
@olavloite olavloite force-pushed the expose-more-connection-api-in-jdbc branch from ba9c863 to 1fac0dc Compare November 9, 2020 07:34
@olavloite olavloite removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Dec 5, 2020
@olavloite olavloite requested a review from a team as a code owner December 5, 2020 09:20
@codecov
Copy link

codecov bot commented Dec 5, 2020

Codecov Report

Merging #255 (cb3ac22) into master (9cef4d5) will decrease coverage by 0.50%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #255      +/-   ##
============================================
- Coverage     71.40%   70.90%   -0.51%     
  Complexity     1104     1104              
============================================
  Files            24       24              
  Lines          3396     3420      +24     
  Branches        525      525              
============================================
  Hits           2425     2425              
- Misses          756      780      +24     
  Partials        215      215              
Impacted Files Coverage Δ Complexity Δ
.../com/google/cloud/spanner/jdbc/JdbcConnection.java 44.01% <0.00%> (-5.72%) 42.00 <0.00> (ø)

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 9cef4d5...cb3ac22. Read the comment docs.

@olavloite olavloite merged commit 697837c into master Dec 5, 2020
@olavloite olavloite deleted the expose-more-connection-api-in-jdbc branch December 5, 2020 09:35
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