Skip to content

Conversation

@zoercai
Copy link
Contributor

@zoercai zoercai commented Apr 22, 2021

Allow users to create an instance using processing_units.
1 node = 1000 processing units. There is currently a default node count used if node count is not specified.
create and update now only update processing units rather than node count, as only one can be specified.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@zoercai zoercai requested review from a team as code owners April 22, 2021 00:53
@zoercai zoercai requested review from leahecole and removed request for a team April 22, 2021 00:53
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/python-spanner API. label Apr 22, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Apr 22, 2021
@zoercai zoercai requested a review from larkee April 22, 2021 00:54
@zoercai zoercai added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 22, 2021
@leahecole
Copy link
Contributor

I think it's adding me as a reviewer because of the synthtool changes in .github/header-checker-lint.yml and those LGTM :)

@zoercai zoercai requested a review from larkee April 26, 2021 07:02
@zoercai zoercai requested a review from larkee April 26, 2021 07:44
Copy link
Contributor

@larkee larkee left a comment

Choose a reason for hiding this comment

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

Just a couple comments to go with the proposed changes to resolve the update issue we discussed 👍

@zoercai zoercai requested a review from larkee May 7, 2021 06:27
@leahecole leahecole removed their request for review June 4, 2021 18:11
@zoercai zoercai removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 22, 2021
@larkee larkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@larkee larkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@larkee larkee merged commit 44aa7cc into googleapis:master Jun 23, 2021
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 googleapis/python-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