Skip to content

doc(bigtable): revamp examples README#6839

Merged
dbolduc merged 3 commits intogoogleapis:mainfrom
dbolduc:bigtable-example-docs-update
Jun 23, 2021
Merged

doc(bigtable): revamp examples README#6839
dbolduc merged 3 commits intogoogleapis:mainfrom
dbolduc:bigtable-example-docs-update

Conversation

@dbolduc
Copy link
Copy Markdown
Member

@dbolduc dbolduc commented Jun 22, 2021

Fixes #6818


This change is Reviewable

@dbolduc dbolduc requested a review from a team June 22, 2021 16:36
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 22, 2021
@product-auto-label product-auto-label Bot added the api: bigtable Issues related to the Bigtable API. label Jun 22, 2021
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 1283de65df20c68436b3850768664362d31914e4

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

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 22, 2021

Codecov Report

Merging #6839 (abada1b) into main (43cc041) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6839      +/-   ##
==========================================
- Coverage   95.03%   95.03%   -0.01%     
==========================================
  Files        1264     1264              
  Lines      113312   113312              
==========================================
- Hits       107691   107685       -6     
- Misses       5621     5627       +6     
Impacted Files Coverage Δ
...e/cloud/spanner/testing/cleanup_stale_instances.cc 42.42% <0.00%> (-12.13%) ⬇️
...bigtable/examples/bigtable_hello_instance_admin.cc 81.31% <0.00%> (-2.20%) ⬇️
...cloud/pubsub/internal/subscription_session_test.cc 97.78% <0.00%> (-0.25%) ⬇️
...sub/internal/batching_publisher_connection_test.cc 97.58% <0.00%> (-0.21%) ⬇️
google/cloud/grpc_error_delegate.cc 100.00% <0.00%> (+4.16%) ⬆️

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 43cc041...abada1b. Read the comment docs.

@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 96ab255ca97c0696387bf9784f638dce4b9084bf

ℹ️ 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.

A few suggestions below.

@@ -1,208 +1,143 @@
# Cloud Bigtable C++ Client Quick Start Code Samples
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You need some kind of H1 type title, no?

After configuring gRPC, you can run the examples using:
| Target | Description |
| ---------------------------------------- | ----------------------------------- |
| `./bigtable_hello_table_admin` | Demonstration of basic operations |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

s/Demonstration/Demonstrates/ maybe?

## Credentials

This sample showcases reading and writing operations:
The following samples demonstrate use of [Authentication](https://cloud.google.com/bigtable/docs/authentication) and [Access Control]((https://cloud.google.com/bigtable/docs/access-control)) for Bigtable. More details on the samples can be found [here](https://googleapis.dev/cpp/google-cloud-bigtable/latest/bigtable-samples-grpc-credentials.html).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For long links like these consider using reference style links

Copy link
Copy Markdown
Member Author

@dbolduc dbolduc left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 2 files reviewed, 2 unresolved discussions (waiting on @coryan)


google/cloud/bigtable/examples/README.md, line 98 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

s/Demonstration/Demonstrates/ maybe?

I think what I have is clunky. But it is self-consistent. The descriptions all say what the code is (not what the code does).

I am going to leave it as it is, but perhaps this is evidence that a table was not the best data structure


google/cloud/bigtable/examples/README.md, line 127 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

For long links like these consider using reference style links

Fixed.

@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: abada1bea66d1bdf6f7c31946fa4a9e5831a3d12

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

@dbolduc dbolduc merged commit f6e4b11 into googleapis:main Jun 23, 2021
@dbolduc dbolduc deleted the bigtable-example-docs-update branch June 23, 2021 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Update Bigtable examples README.md

3 participants