rhizome-ai commented on pull request #87:
URL: https://github.com/apache/incubator-age/pull/87#issuecomment-875598823
Thank you, Shoaib. Thanks to good comments from you and Josh, I could
easily spot important points.
So I reviewed some alternatives, and revised the structure and API of Python
driver.
1. Type converter was fixed to type extension api of Psycopg2. (Psycopg2 is
widely used among Python developers, so I don't think this tight coupling will
be a big problem.) And I added a usage sample (Jupyter note) for developers
familiar with Psycopg2.
2. The execution API is simplified to match the characteristics of the
Psycopg2 transaction API. The 'queryCypher' was removed and merged into
execCypher.
In using Psycopg driver,
- You must explicitly commit after executing the data changing statements.
- When an error occurred while a statement is executed (even though data
retrieving), the session is blocked before rollback called.
3. I added **numeric annotation**, **multiple column return**, **multiple
hop paths**.
Refactoring of Golang driver will be made next week. Due to the
characteristics of PG drivers that follow Golang's sql standard, different
approaches are required from Python.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@age.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
