Skip to content

Conversation

@thiagotnunes
Copy link
Contributor

From the next major release we will be only supporting Java 8+.

From the next major release we will be only supporting Java 8+
@thiagotnunes thiagotnunes requested review from a team as code owners March 11, 2021 02:57
@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • .github/workflows/ci.yaml
  • .kokoro/nightly/java7.cfg
  • .kokoro/presubmit/java7.cfg
  • CONTRIBUTING.md

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 11, 2021
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/java-spanner API. label Mar 11, 2021
@thiagotnunes thiagotnunes added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 11, 2021
@thiagotnunes
Copy link
Contributor Author

This should only be merged when we are planning the next major bump (for CMEK, #666).

@codecov
Copy link

codecov bot commented Mar 18, 2021

Codecov Report

Merging #946 (a3bece7) into master (32fdd60) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #946      +/-   ##
============================================
- Coverage     85.24%   85.21%   -0.03%     
+ Complexity     2660     2570      -90     
============================================
  Files           146      146              
  Lines         14404    14159     -245     
  Branches       1395     1313      -82     
============================================
- Hits          12278    12066     -212     
- Misses         1542     1544       +2     
+ Partials        584      549      -35     
Impacted Files Coverage Δ Complexity Δ
...a/com/google/cloud/spanner/DatabaseClientImpl.java 76.69% <0.00%> (-7.77%) 21.00% <0.00%> (ø%)
.../cloud/spanner/spi/v1/SpannerErrorInterceptor.java 52.94% <0.00%> (-7.06%) 3.00% <0.00%> (ø%)
...ud/spanner/connection/ClientSideStatementImpl.java 48.88% <0.00%> (-1.12%) 9.00% <0.00%> (ø%)
...oogle/cloud/spanner/PartitionedDmlTransaction.java 82.02% <0.00%> (-0.59%) 14.00% <0.00%> (-1.00%)
...ain/java/com/google/cloud/spanner/SessionPool.java 88.73% <0.00%> (-0.53%) 71.00% <0.00%> (-2.00%)
...m/google/cloud/spanner/spi/v1/GapicSpannerRpc.java 81.41% <0.00%> (-0.48%) 71.00% <0.00%> (-12.00%)
...e/cloud/spanner/connection/CredentialsService.java 90.00% <0.00%> (-0.48%) 8.00% <0.00%> (ø%)
...m/google/cloud/spanner/TransactionManagerImpl.java 87.03% <0.00%> (-0.47%) 21.00% <0.00%> (ø%)
.../com/google/cloud/spanner/AbstractReadContext.java 86.54% <0.00%> (-0.33%) 42.00% <0.00%> (-6.00%)
.../admin/instance/v1/stub/GrpcInstanceAdminStub.java 96.94% <0.00%> (-0.24%) 21.00% <0.00%> (-1.00%)
... and 23 more

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 32fdd60...a3bece7. Read the comment docs.

@thiagotnunes thiagotnunes removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 18, 2021
@thiagotnunes thiagotnunes merged commit 7af1951 into master Mar 18, 2021
@thiagotnunes thiagotnunes deleted the java-8 branch March 18, 2021 04:58
@release-please release-please bot mentioned this pull request Mar 18, 2021
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 API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants