Skip to content

Update sql guide#222

Open
cpcgoogle wants to merge 2 commits into
GoogleCloudPlatform:mainfrom
cpcgoogle:update-sql-guide
Open

Update sql guide#222
cpcgoogle wants to merge 2 commits into
GoogleCloudPlatform:mainfrom
cpcgoogle:update-sql-guide

Conversation

@cpcgoogle

Copy link
Copy Markdown

Add a reference to the SQL for application best practices documentation.

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request updates the Bigtable SQL guide to recommend using SQL functions for data manipulation before implementing custom client-side logic, and adds a link to the application development guide. The reviewer suggested fixing a double-space typo and using more descriptive link text to improve accessibility and readability.

### Functions and operators

Bigtable offers a wide range of [SQL functions](https://docs.cloud.google.com/bigtable/docs/reference/sql/functions-all) and [operators](https://docs.cloud.google.com/bigtable/docs/reference/sql/operators) and [conditional expressions](https://docs.cloud.google.com/bigtable/docs/reference/sql/conditional_expressions).
Bigtable offers a wide range of [SQL functions](https://docs.cloud.google.com/bigtable/docs/reference/sql/functions-all) and [operators](https://docs.cloud.google.com/bigtable/docs/reference/sql/operators) and [conditional expressions](https://docs.cloud.google.com/bigtable/docs/reference/sql/conditional_expressions). When developing an application using Bigtable, you should always consider if the data manipulation can be performed using a SQL function before trying to develop the logic in the client application. For best practices on using Bigtable SQL from an application, see this [guide](https://docs.cloud.google.com/bigtable/docs/googlesql-develop-apps) that explains how to create the client, prepare the query, and read the results.

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.

medium

There is a double space in on using. Additionally, using "this [guide]" as link text is less descriptive. It is better to use more descriptive link text for improved readability and accessibility.

Suggested change
Bigtable offers a wide range of [SQL functions](https://docs.cloud.google.com/bigtable/docs/reference/sql/functions-all) and [operators](https://docs.cloud.google.com/bigtable/docs/reference/sql/operators) and [conditional expressions](https://docs.cloud.google.com/bigtable/docs/reference/sql/conditional_expressions). When developing an application using Bigtable, you should always consider if the data manipulation can be performed using a SQL function before trying to develop the logic in the client application. For best practices on using Bigtable SQL from an application, see this [guide](https://docs.cloud.google.com/bigtable/docs/googlesql-develop-apps) that explains how to create the client, prepare the query, and read the results.
Bigtable offers a wide range of [SQL functions](https://docs.cloud.google.com/bigtable/docs/reference/sql/functions-all) and [operators](https://docs.cloud.google.com/bigtable/docs/reference/sql/operators) and [conditional expressions](https://docs.cloud.google.com/bigtable/docs/reference/sql/conditional_expressions). When developing an application using Bigtable, you should always consider if the data manipulation can be performed using a SQL function before trying to develop the logic in the client application. For best practices on using Bigtable SQL from an application, see the [guide on developing applications with GoogleSQL](https://docs.cloud.google.com/bigtable/docs/googlesql-develop-apps) that explains how to create the client, prepare the query, and read the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant