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.

Sign up for GitHub

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

Jump to bottom

Graph Support for Python Connector #3

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

Graph Support for Python Connector #3

wants to merge 166 commits into from

Conversation

TobiasHafner
Copy link

Features

  1. Graph Results:
    • Implemented deserialization support for graph results, including edges, nodes, and paths. Those are now returned as well.
  2. Unit Tests:
    • Added a unit test to ensure graph results are returned correctly.

gartens added 30 commits January 23, 2024 10:32
@gartens
WIP: Start driver for the new gRPC based interface of Polypheny
fdd5418
@gartens
WIP: Add requirements file
807676a
@gartens
WIP: Various improvements and reformat
5cafbae
@gartens
WIP: Add test for PEP
897ca25
@gartens
WIP: Add setup.py
edc7671
@gartens
WIP: Remove comment
0c62a2c 
fetch_size is needed for unprepared statements
@gartens
WIP: Ignore IDEA folder
fc7af44
@gartens
WIP: 90% test coverage
f545137
@gartens
WIP: make mql db.find() work
1072def
@gartens
WIP: Update protofiles
db1cbf0
@gartens
WIP: Remove obsolete workaround & add explicit close of self.chan on …
2a3ceef 
…error
@gartens
WIP: Add test and extend README on how to run tests with coverage
23dd30c
@gartens
Connection works again
02a8412
@gartens
Commit, Rollback and Disconnect
19b64ab
@gartens
All test pass again
e1c718b
@gartens
Bring test coverage back to >90%
d242c3a
@gartens
Remove unused import & streamline connect RPC with other RPC calls
b9a788e
@gartens
Format and fix various warnings,problems etc. found with PyCharm
9d826f7
@gartens
Rename table for consistency with other tests
fb728de
@gartens
Send the initial fetch_size for each fetch request
bc431f5
@gartens
Rewrite tests in test_conn to no longer use emp table
39d2ca9
@gartens
Streamline the execute function and properly handle results of DDLs/DMLs
15cc9bb
@gartens
Move assert back into the main path
0d720c0
@gartens
Replace emp usage in test_pep with data created by the tests
d563481
@gartens
Prevent disconnect message when an error during connect occurs
2de2a3e
@gartens
Add two assertions for correct rowcount of DMLs
ca3a4e2
@gartens
Sync protobuf files
9dbd909
@gartens
Sync protobuf files
6891192
@gartens
Initialize con early to prevent errors during destruction
10bb0e0
@gartens
Add Transport class to wrap connection specific information
7713bf0
datomo and others added 26 commits May 16, 2024 01:57
@datomo
added github system env variable
a5909b6
@gartens
Convert decimals to integers if possible
ae2ba4d
@vogti
Update readme
f3d6243
@vogti
Adjust workflow
ecf550a
@vogti
Fix publish workflow
69e37bd
@vogti
Delete protoc file
de15aed
@gartens
Add license header
ac1fa7c
@vogti
Use prism api version 1.5 from pypi
ad1b763
@vogti
Bump prism-api version to 1.6
5eeffc4
@vogti
Use setuptools
e9c3b3d
@vogti
Explicitly set version=0.1 in tests
42bdf7a
@vogti
Change variable to RELEASE_VERSION
9dc74b0
@vogti
Bump prism-api version to 1.9
3092291
@vogti
Adjust publish workflow
549b9a7
@vogti
Fix path in workflow
efe32aa
@hennlo
added setup info
f716bf5
@hennlo
changed version
4fe2d12
@hennlo
changed test structure
9a5d735
@hennlo
updated readme
5e66ed1
Add graph result case
f06ab1b
Add node and edge deserialization
9a599d9
Add tests
794fde3
Add graph demo
4a9d8fd
Update api version
1ace3f1
Bump api version
bbc6e70
Remove unused code
fff878e
@TobiasHafner TobiasHafner requested a review from vogti July 31, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vogti vogti Awaiting requested review from vogti

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@TobiasHafner @gartens @vogti @datomo @hennlo