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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(docs): delete obsolete code samples #386

Merged
merged 8 commits into from
Jun 14, 2023

Conversation

meredithslota
Copy link
Contributor

@meredithslota meredithslota commented Jun 6, 2023

The "hello world" example for Go is here: https://github.com/GoogleCloudPlatform/golang-samples/tree/main/bigtable/helloworld and this code sample references several unmaintained repos and uses OpenCensus (we are encouraging OpenTelemetry these days).

The HBase quickstart in Java is here: https://github.com/GoogleCloudPlatform/cloud-bigtable-examples/tree/main/java/quickstart so removing the outdated duplicate quickstart code.

The "hello world" example for Go is here: https://github.com/GoogleCloudPlatform/golang-samples/tree/main/bigtable/helloworld and this code sample references several unmaintained repos and uses OpenCensus (we are encouraging OpenTelemetry these days).
@product-auto-label product-auto-label bot added api: bigtable Issues related to the GoogleCloudPlatform/cloud-bigtable-examples API. samples Issues that are directly related to samples. labels Jun 6, 2023
@kolea2
Copy link
Collaborator

kolea2 commented Jun 6, 2023

adding @cshaff0524 to take a quick look

@meredithslota
Copy link
Contributor Author

Discussed with @cshaff0524 in chat and confirmed it's ok to move forward with this. Thanks!

@meredithslota meredithslota merged commit 36db8e7 into main Jun 14, 2023
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 GoogleCloudPlatform/cloud-bigtable-examples API. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants