Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

samples: add code samples for PostgreSql dialect#836

Merged
asthamohta merged 31 commits intogoogleapis:mainfrom
rajatbhatta:pg-samples
Oct 26, 2022
Merged

samples: add code samples for PostgreSql dialect#836
asthamohta merged 31 commits intogoogleapis:mainfrom
rajatbhatta:pg-samples

Conversation

@rajatbhatta
Copy link
Copy Markdown
Contributor

No description provided.

@snippet-bot
Copy link
Copy Markdown

snippet-bot bot commented Oct 4, 2022

Here is the summary of changes.

You are about to add 44 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. api: spanner Issues related to the googleapis/python-spanner API. samples Issues that are directly related to samples. labels Oct 4, 2022
@rajatbhatta rajatbhatta added the snippet-bot:force-run Force snippet-bot runs its logic label Oct 6, 2022
@snippet-bot snippet-bot bot removed the snippet-bot:force-run Force snippet-bot runs its logic label Oct 6, 2022
@rajatbhatta rajatbhatta marked this pull request as ready for review October 11, 2022 05:12
@rajatbhatta rajatbhatta requested a review from a team as a code owner October 11, 2022 05:12
@rajatbhatta rajatbhatta requested review from a team, asthamohta and loferris October 11, 2022 05:12
@rajatbhatta rajatbhatta marked this pull request as draft October 11, 2022 12:38
@rajatbhatta rajatbhatta marked this pull request as ready for review October 11, 2022 14:03
@rajatbhatta rajatbhatta enabled auto-merge (squash) October 12, 2022 07:22
@rajatbhatta rajatbhatta disabled auto-merge October 12, 2022 14:14
# format = isort + black
#


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.

nit:remove extra line

OPERATION_TIMEOUT_SECONDS = 240


# [START spanner_postgresql_create_instance]
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.

A pg instance is not required separately. You can reuse the same instance as google sql instance, and delete this test

database=database_name,
statements=[
"""CREATE TABLE Singers (
SingerId bigint NOT NULL,
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.

nit: can you format this?

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.

Also can you adda comment that pg is case insensitive or rename it to lower case

@asthamohta asthamohta self-requested a review October 26, 2022 13:35
@asthamohta asthamohta added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Oct 26, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 26, 2022
@asthamohta asthamohta enabled auto-merge (squash) October 26, 2022 16:47
@asthamohta asthamohta added the cla: yes This human has signed the Contributor License Agreement. label Oct 26, 2022
@asthamohta asthamohta merged commit fbb1440 into googleapis:main Oct 26, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/python-spanner API. cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples. size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants