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 Python Driver to use the new Prism Interface #2

Open
wants to merge 161 commits into
base: master
Choose a base branch
from

Conversation

gartens
Copy link

@gartens gartens commented Apr 19, 2024

This updates the Python driver to use the new Prism Interface of Polypheny. The driver is DBI 2.0 compatible except for functionality related to the type_code field of the cursor description attribute. See the tests in test_pep.py for more information.

Multi-model

The executeany method allows to use languages other than SQL. The driver can handle documents results in addition to relational results.

Breaking changes

The signature of the connect method has changed.

polypheny.connect('127.0.0.1', 20590, user='...', password='...')

becomes

polypheny.connect(('127.0.0.1', 20590), username='...', password='...', transport='plain')

If you are using the same host and user as Polypheny (and are not on Windows), you can take advantage of the new UNIX socket support of the interface and connect without any parameters at all:

polypheny.connect()

fetch_size is needed for unprepared statements
@gartens
Copy link
Author

gartens commented May 16, 2024

This file was not added by me.

@vogti vogti force-pushed the proto-without-grpc branch from 1df957c to ff2ce0b Compare May 17, 2024 11:21
@vogti vogti force-pushed the proto-without-grpc branch from ff2ce0b to ad1b763 Compare May 17, 2024 11:22
@vogti vogti force-pushed the proto-without-grpc branch from d6175cd to 85c9b17 Compare May 17, 2024 12:33
@vogti vogti force-pushed the proto-without-grpc branch from 85c9b17 to 42bdf7a Compare May 17, 2024 12:35
@vogti vogti force-pushed the proto-without-grpc branch from 4e819b7 to 932fa8b Compare May 17, 2024 13:34
@vogti vogti force-pushed the proto-without-grpc branch 2 times, most recently from d84b4f2 to e4f16c1 Compare May 17, 2024 14:03
@vogti vogti force-pushed the proto-without-grpc branch from e4f16c1 to 549b9a7 Compare May 17, 2024 14:07
@vogti vogti force-pushed the proto-without-grpc branch from 351814f to efe32aa Compare May 17, 2024 14:43
@gartens gartens marked this pull request as ready for review October 10, 2024 14:51
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.

4 participants