Skip to content

Conversation

@olavloite
Copy link
Collaborator

The AbstractLazyInitializer#initialize() method should be protected.

@olavloite olavloite requested a review from a team as a code owner October 21, 2020 13:30
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 21, 2020
@olavloite
Copy link
Collaborator Author

@thiagotnunes It would be good to include this change in the release with the other breaking changes.

@olavloite olavloite requested a review from a team as a code owner October 21, 2020 13:42
@codecov
Copy link

codecov bot commented Oct 21, 2020

Codecov Report

Merging #536 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #536   +/-   ##
=========================================
  Coverage     84.04%   84.04%           
+ Complexity     2540     2537    -3     
=========================================
  Files           140      140           
  Lines         13891    13891           
  Branches       1329     1329           
=========================================
  Hits          11675    11675           
  Misses         1672     1672           
  Partials        544      544           
Impacted Files Coverage Δ Complexity Δ
.../google/cloud/spanner/AbstractLazyInitializer.java 100.00% <ø> (ø) 5.00 <0.00> (ø)
...m/google/cloud/spanner/LazySpannerInitializer.java 50.00% <ø> (ø) 1.00 <0.00> (ø)
.../google/cloud/spanner/SpannerExceptionFactory.java 80.48% <0.00%> (-2.44%) 39.00% <0.00%> (-1.00%)
...om/google/cloud/spanner/TransactionRunnerImpl.java 86.91% <0.00%> (+0.52%) 9.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 768c19d...0b24d1c. Read the comment docs.

@thiagotnunes thiagotnunes merged commit 5c4c8c5 into master Oct 22, 2020
@thiagotnunes thiagotnunes deleted the make-lazy-init-protected branch October 22, 2020 00:42
@release-please release-please bot mentioned this pull request Oct 22, 2020
ansh0l pushed a commit to ansh0l/java-spanner that referenced this pull request Nov 10, 2022
This is an auto-generated regeneration of the .pb.go files by
cloud.google.com/go/internal/gapicgen. Once this PR is submitted, genbot will
update the corresponding PR to depend on the newer version of go-genproto, and
assign reviewers. Whilst this or any regen PR is open in go-genproto, genbot
will not create any more regeneration PRs. If all regen PRs are closed,
gapicgen will create a new set of regeneration PRs once per night.

If you have been assigned to review this PR, please:

- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship. That will prompt
genbot to assign reviewers to the google-cloud-go PR.

Corresponding google-cloud-go PR: googleapis/google-cloud-go#3719

Changes:

feat(spanner/admin/database): add CMEK fields to backup and database
  PiperOrigin-RevId: 358725120
  Source-Link: googleapis/googleapis@f829b13

chore: release gapic-generator-java v0.0.21 with gapic_metadata.json support
  Committer: @miraleung
  PiperOrigin-RevId: 358516065
  Source-Link: googleapis/googleapis@af8f71d

chore: update rules_gapic to v0.4.0 (fixes build_gen for Java packages)
  Committer: @miraleung
  PiperOrigin-RevId: 358464992
  Source-Link: googleapis/googleapis@ab95a4f

chore: update maps/routespreferred Gapic schema to 2.0
  Committer: @miraleung
  PiperOrigin-RevId: 358452585
  Source-Link: googleapis/googleapis@fce32c0

feat(area120/tables): Added ListWorkspaces, GetWorkspace, BatchDeleteRows APIs.
  PiperOrigin-RevId: 358406243
  Source-Link: googleapis/googleapis@151bfba
rajatbhatta pushed a commit to rajatbhatta/java-spanner that referenced this pull request Nov 17, 2022
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/d9079f8e-3fa1-44e6-b6d8-65a7341e9be2/targets

- [ ] To automatically regenerate this PR, check this box. (May take up to 24 hours.)

Source-Link: googleapis/synthtool@5d3c52f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

3 participants