Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update DBAPI to QueryService #58

Merged
merged 5 commits into from
Nov 13, 2024
Merged

Update DBAPI to QueryService #58

merged 5 commits into from
Nov 13, 2024

Conversation

vgvoleg
Copy link
Collaborator

@vgvoleg vgvoleg commented Oct 31, 2024

DBAPI was made as a separate package with runs over QueryService instead of TableService
https://github.com/ydb-platform/ydb-python-dbapi

This PR updates ydb dialect to use new DBAPI

@vgvoleg vgvoleg changed the title Move DBAPI to ext repo Draft: Move DBAPI to ext repo Oct 31, 2024
@vgvoleg vgvoleg force-pushed the move_dbapi_to_ext_repo branch 8 times, most recently from 8d8eac2 to 61a6e32 Compare November 2, 2024 12:26
@vgvoleg vgvoleg changed the title Draft: Move DBAPI to ext repo Update DBAPI to QueryService Nov 2, 2024
@vgvoleg vgvoleg force-pushed the move_dbapi_to_ext_repo branch from 61a6e32 to 98c3318 Compare November 2, 2024 12:30
@vgvoleg vgvoleg force-pushed the move_dbapi_to_ext_repo branch from 98c3318 to e0233e3 Compare November 5, 2024 09:45
Copy link
Contributor

@LuckySting LuckySting left a comment

Choose a reason for hiding this comment

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

praise: Good job!

ydb_sqlalchemy/sqlalchemy/__init__.py Outdated Show resolved Hide resolved
@vgvoleg vgvoleg force-pushed the move_dbapi_to_ext_repo branch from 694759d to 512b1eb Compare November 13, 2024 08:36
@vgvoleg vgvoleg merged commit 5a14a77 into main Nov 13, 2024
6 checks passed
@vgvoleg vgvoleg deleted the move_dbapi_to_ext_repo branch November 13, 2024 09:55
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.

2 participants