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

feat: DB-API driver + unit tests - #160

Merged
larkee merged 14 commits into
googleapis:masterfrom
MaxxleLLC:spanner-dbapi
Nov 11, 2020
Merged

feat: DB-API driver + unit tests#160
larkee merged 14 commits into
googleapis:masterfrom
MaxxleLLC:spanner-dbapi

Conversation

@mf2199

@mf2199mf2199 commented Oct 30, 2020

Copy link
Copy Markdown
Contributor

The DB-API driver refactored from python-spanner-django repository, prior to its beta-release.

Some additional changes have been made to skip unit tests found incompatible with Python 2.

@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Oct 30, 2020
@mf2199mf2199 added api: spanner Issues related to the googleapis/python-spanner API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Oct 31, 2020
@c24tc24t removed the cla: yes This human has signed the Contributor License Agreement. label Nov 2, 2020
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Nov 2, 2020
@mf2199
mf2199 marked this pull request as ready for review November 2, 2020 19:42
@mf2199
mf2199 requested a review from a teamNovember 2, 2020 19:42
c24t
c24t approved these changes Nov 2, 2020

@c24tc24t 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.

LGTM, we'll need a PR in https://github.com/googleapis/python-spanner-django to remove spanner_dbapi and then coordinate releasees. We can follow up with tests for 2.x.

@skuruppu
skuruppu requested a review from larkeeNovember 4, 2020 00:14

@larkeelarkee 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.

LGTM 👍 Just one concern about NUMERIC loss of precision.

Comment threadgoogle/cloud/spanner_dbapi/_helpers.py Outdated
Comment threadgoogle/cloud/spanner_dbapi/cursor.py
Comment threadgoogle/cloud/spanner_dbapi/parse_utils.py Outdated
@c24tc24t mentioned this pull request Nov 10, 2020
@larkeelarkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 11, 2020
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 11, 2020
@larkee
larkee merged commit 2493fa1 into googleapis:masterNov 11, 2020
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Mar 6, 2026
BREAKING CHANGE: DBAPI code was moved into python-spanner in
googleapis/python-spanner#160. This change removes it
from this repo and bumps the dependency on python-spanner to 2.0.0, the first
released version to include DBAPI.
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spannerIssues related to the googleapis/python-spanner API.cla: yesThis human has signed the Contributor License Agreement.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@mf2199@larkee@c24t@yoshi-kokoro