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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regenerate spanner client #6731

Merged
merged 3 commits into from Nov 12, 2019
Merged

Regenerate spanner client #6731

merged 3 commits into from Nov 12, 2019

Conversation

yoshi-automation
Copy link
Contributor

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
Timeout parameters are updated to reflect timeout changes to
spanner_gapic.yaml in googleapis/googleapis commit 6b12595.
synthtool incorrectly generated code with a "com.google.iam.v1." prefix
for DatabaseName and InstanceName.
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
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants