Skip to content

feat(spanner): add generated admin APIs#7183

Merged
devbww merged 1 commit intogoogleapis:mainfrom
devbww:generated-admin
Aug 17, 2021
Merged

feat(spanner): add generated admin APIs#7183
devbww merged 1 commit intogoogleapis:mainfrom
devbww:generated-admin

Conversation

@devbww
Copy link
Copy Markdown
Contributor

@devbww devbww commented Aug 17, 2021

Configure generation of the Spanner database/instance admin APIs:

  • in google::cloud::spanner_admin namespace
  • in google/cloud/spanner/admin include path

This will allow them to co-exist with the hand-wrought APIs until
the latter can be deprecated and eventually removed.

The actual merge of the generated code will be in followup PRs.


This change is Reviewable

Configure generation of the Spanner database/instance admin APIs:
- in `google::cloud::spanner_admin` namespace
- in `google/cloud/spanner/admin` include path

This will allow them to co-exist with the hand-wrought APIs until
the latter can be deprecated and eventually removed.

The actual merge of the generated code will be in followup PRs.
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the Spanner API. label Aug 17, 2021
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 17, 2021
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 015180530ae0a1f4c3498b9813486b6ff66eac50

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

Copy link
Copy Markdown
Contributor

@coryan coryan left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @devbww)

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 17, 2021

Codecov Report

Merging #7183 (0151805) into main (ff8d468) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7183   +/-   ##
=======================================
  Coverage   94.44%   94.44%           
=======================================
  Files        1310     1310           
  Lines      113526   113526           
=======================================
+ Hits       107219   107223    +4     
+ Misses       6307     6303    -4     
Impacted Files Coverage Δ
google/cloud/storage/internal/complex_option.h 90.90% <0.00%> (-9.10%) ⬇️
...le/cloud/storage/internal/curl_download_request.cc 89.21% <0.00%> (-0.38%) ⬇️
google/cloud/completion_queue_test.cc 96.95% <0.00%> (-0.20%) ⬇️
google/cloud/pubsub/subscriber_connection_test.cc 97.90% <0.00%> (+0.69%) ⬆️
google/cloud/grpc_error_delegate.cc 100.00% <0.00%> (+4.16%) ⬆️
...e/cloud/spanner/testing/cleanup_stale_instances.cc 54.54% <0.00%> (+12.12%) ⬆️

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 ff8d468...0151805. Read the comment docs.

@devbww devbww marked this pull request as ready for review August 17, 2021 17:43
@devbww devbww requested a review from a team August 17, 2021 17:43
@devbww devbww merged commit e59db51 into googleapis:main Aug 17, 2021
@devbww devbww deleted the generated-admin branch August 17, 2021 17:48
devbww added a commit to devbww/google-cloud-cpp that referenced this pull request Aug 31, 2021
Revert googleapis#7183, googleapis#7184, and googleapis#7188 prior to the monthly release, as they
implement a partially-complete but user-visible feature.  This will
avoid any confusion about the availability of the feature.
devbww added a commit that referenced this pull request Aug 31, 2021
Revert #7183, #7184, and #7188 prior to the monthly release, as they
implement a partially-complete but user-visible feature.  This will
avoid any confusion about the availability of the feature.
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 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.

4 participants