8000 Regenerate spanner client by yoshi-automation · Pull Request #6731 · googleapis/google-cloud-java · GitHub
[go: up one dir, main page]
Skip to content

Regenerate spanner client #6731

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.

Sign up for GitHub

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

Jump to bottom
Merged
skuruppu merged 3 commits into from
Nov 12, 2019
Merged

Regenerate spanner client #6731

skuruppu merged 3 commits into from
Nov 12, 2019

Conversation

@yoshi-automation
Copy link
Contributor
@yoshi-automation yoshi-automation commented Nov 12, 2019

This PR was generated using Autosynth. 🌈

Log from Synthtool 
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/google-cloud-clients/google-cloud-spanner/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:545c758c76c3f779037aa259023ec3d1ef2d57d2c8cd00a222cb187d63ceac5e
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/spanner/artman_spanner.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/java.
synthtool > Running generator for google/spanner/admin/database/artman_spanner_admin_database.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/java.
synthtool > Running generator for google/spanner/admin/instance/artman_spanner_admin_instance.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/java.
synthtool > Running java formatter on 190 files
synthtool > Running java formatter on 1 files
synthtool > Running java formatter on 80 files
synthtool > Running java formatter on 1 files
synthtool > Running java formatter on 25 files
synthtool > Running java formatter on 1 files
synthtool > Running java formatter on 32 files
synthtool > Cleaned up 0 temporary directories.
synthtool > Wrote metadata to synth.metadata.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 12, 2019
@skuruppu
Fix testSpannerDefaultRetrySettings test.
2104ade 
Timeout parameters are updated to reflect timeout changes to
spanner_gapic.yaml in googleapis/googleapis commit 6b12595.
@skuruppu
Remove incorrect "com.google.iam.v1." prefix.
cdf6f28 
synthtool incorrectly generated code with a "com.google.iam.v1." prefix
for DatabaseName and InstanceName.
skuruppu
skuruppu approved these changes Nov 12, 2019
View reviewed changes
Copy link
@skuruppu skuruppu left a comment

Choose a reason for hiding this comment

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

Approved but following up with @kolea2 offline on how to fix the underlying issue so that the code generation doesn't include the spurious "com.google.iam.v1." prefix.

@codecov
Copy link
codecov bot commented Nov 12, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #6731   +/-   ##
=========================================
  Coverage     46.37%   46.37%           
  Complexity    28024    28024           
=========================================
  Files          2614     2614           
  Lines        288317   288317           
  Branches      33810    33810           
=========================================
  Hits         133715   133715           
  Misses       144336   144336           
  Partials      10266    10266

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 66a3423...cdf6f28. Read the comment docs.

@skuruppu skuruppu merged commit e84e89d into master Nov 12, 2019
@skuruppu skuruppu deleted the autosynth-spanner branch November 12, 2019 23:38
@chingor13 chingor13 mentioned this pull request Nov 13, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@skuruppu skuruppu skuruppu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yoshi-automation @skuruppu @googlebot
0