Skip to content

Conversation

@IlyaFaer
Copy link
Contributor

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 19, 2021
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/python-spanner API. label Oct 19, 2021
@IlyaFaer IlyaFaer requested review from larkee and skuruppu October 19, 2021 11:33
@IlyaFaer IlyaFaer marked this pull request as ready for review October 19, 2021 11:33
@IlyaFaer IlyaFaer requested a review from a team as a code owner October 19, 2021 11:33
@larkee larkee changed the title feat: emit warning instead of an exception for rowcount property fix: emit warning instead of an exception for rowcount property Oct 21, 2021
@larkee larkee changed the title fix: emit warning instead of an exception for rowcount property fix(db_api): emit warning instead of an exception for rowcount property Oct 21, 2021
@larkee larkee added the automerge Merge the pull request once unit tests and other checks pass. label Oct 21, 2021
@IlyaFaer IlyaFaer added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 21, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 21, 2021
@IlyaFaer
Copy link
Contributor Author

@larkee, only samples failed with timeout errors, not related. I guess your admin access could finish the job here (not to hold off another related PR).

@gcf-merge-on-green
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@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 21, 2021
@IlyaFaer IlyaFaer requested a review from a team as a code owner October 25, 2021 09:47
@larkee larkee added the automerge Merge the pull request once unit tests and other checks pass. label Oct 25, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit 62ff9ae into googleapis:main Oct 26, 2021
@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, 2021
@IlyaFaer IlyaFaer deleted the exception_to_warn branch October 26, 2021 08:11
asthamohta pushed a commit to asthamohta/python-spanner that referenced this pull request Nov 11, 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.

3 participants